RE: [PATCH] Support automatic sequence replication

Zhijie Hou (Fujitsu) <houzj.fnst@fujitsu.com>

From: "Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>

To: "Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>, Amit Kapila <amit.kapila16@gmail.com>, Ajin Cherian <itsajin@gmail.com>

Cc: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>, shveta malik <shveta.malik@gmail.com>, Ashutosh Sharma <ashu.coek88@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>

Date: 2026-03-02T07:58:54Z

Lists: pgsql-hackers

Attachments

v9-0002-Synchronize-sequences-directly-in-REFRESH-SEQUENC.patch (application/octet-stream) patch v9-0002
v9-0001-Support-automatic-sequence-replication.patch (application/octet-stream) patch v9-0001

On Saturday, February 28, 2026 4:41 PM Zhijie Hou (Fujitsu) <houzj.fnst@fujitsu.com> wrote:
> Here is the V8 patch set addressing the previous review comments:
> 
> - For 0001, I noticed that the GetSequence() function added in the patch
>   fetches the local sequence value without any privilege check. This
>   allows the worker to read sequence data even without proper SELECT
>   privilege, which seems unsafe. I've added a SELECT privilege check
>   before fetching the sequence value. Additionally, I've updated several
>   comments, made cosmetic changes, commit message update, and run
>   pgindent on all patches.
> 
> - 0002 includes the changes to synchronize sequences directly in the
>   REFRESH SEQUENCES command

Rebased the patch to silence compile warning due to a recent commit
a2c89835.

Best Regards,
Hou zj

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Don't include wait_event.h in pgstat.h
- 868825aaeb40 19 (unreleased) cited
Don't include proc.h in shm_mq.h
- a2c89835f512 19 (unreleased) cited
Fix unsafe RTE_GROUP removal in simplify_EXISTS_query
- 77c7a17a6e5f 19 (unreleased) cited
Add sequence synchronization for logical replication.
- 5509055d6956 19 (unreleased) cited