Re: ALTER SUBSCRIPTION REFRESH PUBLICATION has default copy_data = true?

From: Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>
To: Peter Smith <smithpb2250(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: ALTER SUBSCRIPTION REFRESH PUBLICATION has default copy_data = true?
Date: 2021-06-02 14:53:30
Message-ID: CALj2ACUQZGqWOfSF0Xpc0-eFWKpmq9-3ewuHuZXx4gv=WrpP8Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jun 2, 2021 at 6:40 AM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> Hi.
>
> The documentation of ALTER SUBSCRIPTION REFRESH PUBLICATION [1] says:
>
> ----------
>
> REFRESH PUBLICATION
>
> Fetch missing table information from publisher. This will start
> replication of tables that were added to the subscribed-to
> publications since the last invocation of REFRESH PUBLICATION or since
> CREATE SUBSCRIPTION.
>
> refresh_option specifies additional options for the refresh operation.
> The supported options are:
>
> copy_data (boolean)
>
> Specifies whether the existing data in the publications that are being
> subscribed to should be copied once the replication starts. The
> default is true. (Previously subscribed --tables are not copied.)
>
> ----------
>
> But I found that default copy_data = true to be unintuitive.
>
> e.g. When I had previously done the CREATE SUBSCRIPTION using
> copy_data = false, then I assumed (wrongly) that the subscription
> default would remain as copy_data = false even when doing the REFRESH
> PUBLICATION.

The fact is that the options copy_data, create_slot and connect are
not stored in the catalog pg_subscription (actually they don't need to
be). Among these the copy_data option can be specified in the ALTER
... SUBSCRIPTION.

> Is that a deliberate functionality, or is it a quirk / bug?

I don't think it's a bug. Maybe adding a note in the docs about the
options which are stored/not stored in the pg_subscription catalog
would help.

With Regards,
Bharath Rupireddy.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Matthias van de Meent 2021-06-02 14:54:06 Re: pg_stat_progress_create_index vs. parallel index builds
Previous Message Peter Eisentraut 2021-06-02 14:52:42 Re: ALTER SUBSCRIPTION REFRESH PUBLICATION has default copy_data = true?