On Mon, 17 Dec 2018 at 12:59, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> Okay, I think that you should add an assertion on
> CheckRelationLockedByMe() as MergeAttributes()'s only caller is
> DefineRelation(). Better safe than sorry later.
Would that not just double up the work that's already done in relation_open()?
--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services