pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Amit Kapila <akapila@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Refactor logical worker synchronization code into a separate fil
Date: Thu, 16 Oct 2025 05:21:42 +0000
Message-ID: <E1v9GQo-001vqo-0t@gemulon.postgresql.org> (raw)

Refactor logical worker synchronization code into a separate file.

To support the upcoming addition of a sequence synchronization worker,
this patch extracts common synchronization logic shared by table sync
workers and the new sequence sync worker into a dedicated file. This
modularization improves code reuse, maintainability, and clarity in the
logical workers framework.

Author: vignesh C <vignesh21@gmail.com>
Author: Hou Zhijie <houzj.fnst@fujitsu.com>
Reviewed-by: shveta malik <shveta.malik@gmail.com>
Reviewed-by: Dilip Kumar <dilipbalaut@gmail.com>
Reviewed-by: Peter Smith <smithpb2250@gmail.com>
Reviewed-by: Hayato Kuroda <kuroda.hayato@fujitsu.com>
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Reviewed-by: Amit Kapila <amit.kapila16@gmail.com>
Discussion: https://postgr.es/m/CAA4eK1LC+KJiAkSrpE_NwvNdidw9F2os7GERUeSxSKv71gXysQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/41c674d2e31e8304a6edbcb5183d326798ba00f6

Modified Files
--------------
src/backend/catalog/pg_subscription.c              |   4 +-
src/backend/replication/logical/Makefile           |   1 +
.../replication/logical/applyparallelworker.c      |   2 +-
src/backend/replication/logical/meson.build        |   1 +
src/backend/replication/logical/syncutils.c        | 187 ++++++++++++++++++++
src/backend/replication/logical/tablesync.c        | 196 +++------------------
src/backend/replication/logical/worker.c           |  22 +--
src/bin/pg_dump/common.c                           |   4 +-
src/bin/pg_dump/pg_dump.c                          |   8 +-
src/bin/pg_dump/pg_dump.h                          |   2 +-
src/include/catalog/pg_subscription_rel.h          |   2 +-
src/include/replication/worker_internal.h          |  14 +-
src/tools/pgindent/typedefs.list                   |   2 +-
13 files changed, 243 insertions(+), 202 deletions(-)



Message-ID: <E1v9GQo-001vqo-0t@gemulon.postgresql.org>
Permalink:  ../E1v9GQo-001vqo-0t@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1v9GQo-001vqo-0t@gemulon.postgresql.org

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: akapila@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Refactor logical worker synchronization code into a separate fil
  In-Reply-To: <E1v9GQo-001vqo-0t@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox