From: | Doruk Yilmaz <doruk(at)mixrank(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Euler Taveira <euler(at)eulerto(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: [Patch] add new parameter to pg_replication_origin_session_setup |
Date: | 2025-07-28 21:13:27 |
Message-ID: | CAMPB6wfgvWjSvKNPoJkRqaL46geRDoL++Pt_3Czc2QNAdpVQHw@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Mar 3, 2025 at 6:39 AM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> To use replication_origin by multiple processes, one must maintain the
> commit order as we do internally by allowing the leader process to
> wait for the parallel worker to finish the commit. See comments atop
> replorigin_session_setup(). Now, we could expose the pid parameter as
> proposed by the patch after documenting the additional requirements,
> but I am afraid that users may directly start using the API without
> following the commit order principle, which can lead to incorrect
> replication. So, isn't it better to do something to avoid the misuse
> of this feature before exposing it?
Wouldn't mentioning/describing needing to follow the commit order
principle on the documentation be enough for this?
It is quite an advanced feature that I don't believe person intending
to use it won't start with reading documentation first.
Is there any updates on the commit? I see that intended commitfest window ended.
Thanks,
Doruk Yılmaz
From | Date | Subject | |
---|---|---|---|
Next Message | Álvaro Herrera | 2025-07-28 21:13:44 | Re: refactor backend type lists |
Previous Message | Masahiko Sawada | 2025-07-28 21:11:39 | Re: POC: enable logical decoding when wal_level = 'replica' without a server restart |