On Tue, Sep 23, 2025 at 3:28 AM Masahiko Sawada <[email protected]> wrote:
>
>
> I've attached the updated patch. It incorporates all comments I got so
> far and implements to lazily disable logical decoding. It's used only
> when the process tries to disable logical decoding during process
> exit.
>
I am resuming the review now. I agree with the discussion of lazily
disabling logical decoding on ERROR or process-exit for temp-slot.
Few initial comments:
1)
I see that on standby too, during proc-exit, we set 'pending_disable'.
But it never resets it, as DisableLogicalDecodingIfNecessary is no-op
on standby. And thus the checkpoint keeps on attempting to reset it
everytime. Do we even need to set it on standby?
Logfile has repeated: 'start completing pending logical decoding
disable request'
2)
+ ereport(LOG,
+ (errmsg("skip disabling logical decoding as during process exit")));
'as' not needed.
thanks
Shveta
