From: | Peter Smith <smithpb2250(at)gmail(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | shveta malik <shveta(dot)malik(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, Shlok Kyal <shlok(dot)kyal(dot)oss(at)gmail(dot)com>, Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: POC: enable logical decoding when wal_level = 'replica' without a server restart |
Date: | 2025-10-23 03:28:05 |
Message-ID: | CAHut+Ps8GEcsrejO1YUmb138a6XunV5uPmvppJ+WKRaFRhJnqw@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Oct 22, 2025 at 8:28 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Wed, Oct 22, 2025 at 2:40 PM shveta malik <shveta(dot)malik(at)gmail(dot)com> wrote:
> >
> > On Fri, Oct 17, 2025 at 11:09 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
> > >
> >
> > I want to discuss the create-publication case, which currently gives
> > this warning:
> >
> > postgres=# create publication pub1 for all tables;
> > WARNING: logical decoding should be allowed to publish logical changes
> > HINT: Before creating subscriptions, set "wal_level" >= "logical" or
> > create a logical replication slot when "wal_level" = "replica".
> > CREATE PUBLICATION
> >
> > But is this warning really necessary during publication creation?
> >
>
> On HEAD, the WARNING/HINT is as follows for the above case:
> WARNING: "wal_level" is insufficient to publish logical changes
> HINT: Set "wal_level" to "logical" before creating subscriptions.
>
> Shouldn't we simply change the HINT to "Set "wal_level" to "logical"
> or create a logical replication slot before creating subscriptions."?
>
I had suggested in a previous review [1-#6b] that perhaps this HINT
wording is overkill now. e.g. I felt now maybe it only needs to say.
HINT: Set "wal_level" >= "replica".
E.g. I thought there is no need to mention "create a logical slot
before creating subscriptions", because creating a typical
subscription (which has a slot) is doing that anyway, so the
publisher's effective_wal_level will increase on the fly, without the
user needing to do anything.
At least, that is what I observed with the experiment shown below:
Indeed, doesn't it mean this entire WARNING is redundant now? e.g.
When wal_level is "minimal", I thought the CREATE PUBLICATION command
would be impossible in the first place. And for wal_level = "replica"
or "logcial", it just works without the user doing anything.
//////////
EXPERIMENT -- Try creating a subscription that will automatically bump
up the effective_wal_level on the Publisher.
==========
On Publisher
==========
test_pub=# show wal_level; show effective_wal_level;
wal_level
-----------
replica
(1 row)
effective_wal_level
---------------------
replica
(1 row)
test_pub=# create publication mypub for all tables;
WARNING: logical decoding must be enabled to publish logical changes
HINT: Before creating subscriptions, set "wal_level" >= "logical" or
create a logical replication slot when "wal_level" = "replica".
CREATE PUBLICATION
==========
On Subscriber:
==========
test_sub=# show wal_level; show effective_wal_level;
wal_level
-----------
replica
(1 row)
effective_wal_level
---------------------
replica
(1 row)
test_sub=# create subscription mysub connection 'dbname=test_pub'
publication mypub;
NOTICE: created replication slot "mysub" on publisher
CREATE SUBSCRIPTION
test_sub=# 2025-10-23 13:56:08.260 AEDT [14453] LOG: logical
replication apply worker for subscription "mysub" has started
2025-10-23 13:56:08.278 AEDT [14455] LOG: logical replication table
synchronization worker for subscription "mysub", table "t1" has
started
2025-10-23 13:56:08.391 AEDT [14455] LOG: logical replication table
synchronization worker for subscription "mysub", table "t1" has
finished
==========
Now, back on the Publisher:
==========
test_pub=# show wal_level; show effective_wal_level;
wal_level
-----------
replica
(1 row)
effective_wal_level
---------------------
logical
(1 row)
Kind Regards,
Peter Smith.
Fujitsu Australia
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2025-10-23 03:33:02 | Re: [Proposal] Adding callback support for custom statistics kinds |
Previous Message | Quan Zongliang | 2025-10-23 03:18:09 | Re: [PATCH] Free memory allocated by waitonlock_error_callback() |