On Thu, Nov 30, 2017 at 10:52 AM, Michael Paquier <michael.paqu...@gmail.com> wrote: > 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.
Thank you Michael-san, I'll submit a latest patch. Regards, -- Masahiko Sawada NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center
