| From: | vignesh C <vignesh21(at)gmail(dot)com> |
|---|---|
| To: | Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> |
| Cc: | 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 07:28:40 |
| Message-ID: | CALDaNm0yjS9WF72=Ny_KKN3zUHMb19OyvxFt=D5+adZE+Cs--w@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, 2 Sept 2026 at 07:44, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
>
> V9 overall looks good. A few small comments:
>
> 1
> ```
> +# run pg_createsubscriber with '--all' with debug verbose logging enabled
> my ($stdout, $stderr) = run_command(
> [
> 'pg_createsubscriber',
> '--verbose',
> + '--verbose',
> ```
>
> The test already had —verbose, adding the option again is redundant.
This was done intentionally to run in debug logging mode to verify the
create publication command includes ALL SEQUENCES or not. Added a
comment on top of test to avoid confusion.
> 2
> ```
> + Create a publication and replication slot for each specified database
> + on the source server. If the <option>--publication</option> option is
> + specified, the user-specified publication is used, which includes the
> + objects defined by that publication. Otherwise,
> ```
>
> This statement doesn’t seem precise to me. --publication might specify a non-existing publication, in which case a new publication will be created, but “the user-specified publication is used” seems to imply that the specified publication must already exist.
Update the documents to clarify this.
The attached v10 version patch has the changes for the same.
Regards,
Vignesh
| Attachment | Content-Type | Size |
|---|---|---|
| v10-0001-Include-sequences-in-publications-created-by-pg_.patch | application/octet-stream | 10.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Xuneng Zhou | 2026-09-09 07:30:22 | Re: Reject WAIT FOR earlier in transaction-snapshot mode |
| Previous Message | David Geier | 2026-09-09 07:22:47 | Re: Add pg_stat_vfdcache view for VFD cache statistics |