| From: | solai v <solai(dot)cdac(at)gmail(dot)com> |
|---|---|
| To: | vignesh C <vignesh21(at)gmail(dot)com> |
| Cc: | Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, Shlok Kyal <shlok(dot)kyal(dot)oss(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Include sequences in publications created by pg_createsubscriber |
| Date: | 2026-09-09 12:42:22 |
| Message-ID: | CAF0whufH456kHDAcd6cY3ALJVFLzzyvEKLSejZAMoqsbq6kq8w@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi all,
Thank you for the updated patch.
I reviewed and tested the patch for including sequences in
publications created by pg_createsubscriber. The patch correctly
changes the internally generated publication from FOR ALL TABLES to
FOR ALL TABLES, ALL SEQUENCES. I also verified the changes in the test
and documentation, including the handling of sequence synchronization
through REFRESH SEQUENCES. I tested the patch with the
pg_createsubscriber TAP tests and the relevant build/tests passed
successfully. I also performed a manual test by converting a physical
standby using pg_createsubscriber and verified from the generated
internal log that the publication command contains both FOR ALL TABLES
and ALL SEQUENCES. The conversion completed successfully. Overall, the
implementation looks correct and the observed behavior is as expected.
The patch looks good to me.
Regards
Solai
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jacob Champion | 2026-09-09 12:43:49 | Re: pg_createsubscriber does not check output_plugin_libraries |
| Previous Message | Jingtang Zhang | 2026-09-09 12:40:23 | Re: [PATCH] Use maintenance_io_concurrency for parallel index builds |