Hello, sorry I didn't notice this email.
On 2019-Sep-26, Michael Paquier wrote:
> This part from commit 773df88 is incorrect:
> + /*
> + * Set the default lock mode for this option. There is no actual way
> + * for a module to enforce it when declaring a custom relation option,
> + * so just use the highest level, which is safe for all cases.
> + */
> + newoption->lockmode = AccessExclusiveLock;
You're right, this was a merge blunder :-( Sorry about that.
> Do you mind if I apply the attached to fix the issue? Or perhaps you
> would prefer fixing the issue yourself? I noted some inconsistencies
> with the rest while on it (please see attached).
Thanks for fixing. It all looks correct to me.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services