On Wed, Nov 29, 2017 at 5:33 AM, Robert Haas <robertmh...@gmail.com> wrote: > On Sun, Nov 26, 2017 at 9:33 PM, Masahiko Sawada <sawada.m...@gmail.com> > wrote: >> Attached latest patch incorporated all comments so far. Please review it. > > I think you only need RelExtLockReleaseAllI() where we currently have > LockReleaseAll(DEFAULT_LOCKMETHOD, ...) not where we have > LockReleaseAll(USER_LOCKMETHOD, ...). That's because relation > extension locks use the default lock method, not USER_LOCKMETHOD.
Latest review is fresh. I am moving this to next CF with "waiting on author" as status. -- Michael