On Tue, Oct 24, 2017 at 1:26 PM, Ivan Kartyshov <i(dot)kartyshov(at)postgrespro(dot)ru> wrote: > Hello. I made some bugfixes and rewrite the patch.
I don't think it's a good idea to deliberately leave the state of the standby different from the state of the master on the theory that it won't matter. I feel like that's something that's likely to come back to bite us.
Giving LockAcquireExtended() an argument that causes some AccessExclusiveLocks not all to be logged also seems pretty ugly, especially because there are no comments whatsoever explaining the rationale.