On Mon, 17 Dec 2018 at 12:59, Michael Paquier <mich...@paquier.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