pgsql: Refactor logical worker synchronization code into a separate fil

From: Amit Kapila <akapila(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Refactor logical worker synchronization code into a separate fil
Date: 2025-10-16 05:21:42
Message-ID: E1v9GQo-001vqo-0t@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

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(at)gmail(dot)com>
Author: Hou Zhijie <houzj(dot)fnst(at)fujitsu(dot)com>
Reviewed-by: shveta malik <shveta(dot)malik(at)gmail(dot)com>
Reviewed-by: Dilip Kumar <dilipbalaut(at)gmail(dot)com>
Reviewed-by: Peter Smith <smithpb2250(at)gmail(dot)com>
Reviewed-by: Hayato Kuroda <kuroda(dot)hayato(at)fujitsu(dot)com>
Reviewed-by: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
Reviewed-by: Amit Kapila <amit(dot)kapila16(at)gmail(dot)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(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Amit Langote 2025-10-16 06:20:29 Re: pgsql: Fix EPQ crash from missing partition directory in EState
Previous Message Amit Langote 2025-10-16 05:03:04 pgsql: Fix EPQ crash from missing partition directory in EState