Re: Include sequences in publications created by pg_createsubscriber

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

In response to

Browse pgsql-hackers by date

  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