Request a Demo
Home   >   mailing lists

Re: POC: enable logical decoding when wal_level = 'replica' without a server restart - Mailing list pgsql-hackers

From shveta malik
Subject Re: POC: enable logical decoding when wal_level = 'replica' without a server restart
Date
Msg-id CAJpy0uCXKPz=kbyBPqnKOqvZm8QAELQDy=LX0RCXt3GnCAa2rA@mail.gmail.com
Whole thread Raw
In response to Re: POC: enable logical decoding when wal_level = 'replica' without a server restart  (Masahiko Sawada <sawada.mshk@gmail.com>)
Responses Re: POC: enable logical decoding when wal_level = 'replica' without a server restart
List pgsql-hackers
Tree view 
On Fri, Dec 19, 2025 at 6:13 AM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
>
> >
>
> I've rebased the patch and addressed the review comments.
>
> Regards,
>

Thanks for the patch.

1)
+ * There could be some transactions that might have started with the old
+ * status, but we don't need to wait for these transactions to complete as
+ * long as they valid XIDs. These transactions will appear in the

they valid --> they have valid

2)
+ * One might think we need to wait for all running transactions, including
+ * read-only transactions, to finish before enabling logical decoding.

Suggestion to change it slightly to make it more clear:
One might think we need to wait for all running transactions,
including those without XIDs and read-only ones, to finish before
enabling logical decoding.
~~

Apart from the above trivial corrections, I have no more comments.
I’ll conclude my review here; the patch LGTM.

thanks
Shveta

pgsql-hackers by date:

Previous
From: Xuneng Zhou
Date:
Subject: Re: eliminate xl_heap_visible to reduce WAL (and eventually set VM on-access)
Next
From: Amit Kapila
Date:
Subject: Re: DOCS - Clarify the publication 'publish_via_partition_root' default value.