Content-Type: text/plain;
	charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3864.400.21\))
Subject: Re: Reduce log level of some logical decoding messages to DEBUG1
From: Chao Li <li.evan.chao@gmail.com>
In-Reply-To: 
 <CAHGQGwHhesQa5KSR76OGxRok0P=_JBstSUGsKUVP-_ZuTjXPQw@mail.gmail.com>
Date: Wed, 1 Apr 2026 15:30:40 +0800
Cc: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>,
 PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <5AFD043F-6542-4BE8-BC46-5F9EF281B3BF@gmail.com>
References: 
 <CAHGQGwGTyHgtD9tyN664x6vQ8Q1G53H7ZUCgBU9_X=nLt3f1QA@mail.gmail.com>
 <CALj2ACX_SSJYKt5od1C2874yeotTO9JewfRY9B=3DYx2daUAMw@mail.gmail.com>
 <CAHGQGwHTuLKTmW5DHtx-5yMuEq4=fc7M+UuvdOVekyvfsi=48Q@mail.gmail.com>
 <CAHGQGwFB+hC97SHDpn1vOHHOq9uM5OZZmt50j7v6Z1JUNtH5zQ@mail.gmail.com>
 <CALj2ACXwe+iVoE50epmfGahYfoeBadgRr8ADvNHshWn0svv9hw@mail.gmail.com>
 <CAHGQGwHUK6gMxF_dO2ei7xFt6ncegACi82BHq4zCMsBuVkxd_w@mail.gmail.com>
 <CAHGQGwHhesQa5KSR76OGxRok0P=_JBstSUGsKUVP-_ZuTjXPQw@mail.gmail.com>
To: Fujii Masao <masao.fujii@gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/5AFD043F-6542-4BE8-BC46-5F9EF281B3BF%40gmail.com>
Precedence: bulk


> On Apr 1, 2026, at 14:48, Fujii Masao <masao.fujii@gmail.com> wrote:
>=20
> On Wed, Mar 25, 2026 at 2:54=E2=80=AFPM Fujii Masao =
<masao.fujii@gmail.com> wrote:
>>> Perhaps, we could wait
>>> for some time to hear from others.
>>=20
>> +1.
>>=20
>> If there are no further comments in the next few days, and given the =
limited
>> time before feature freeze, I'm inclined to at least apply the 0001 =
patch
>> (which simply lowers the log level to DEBUG1).
>=20
> Since there were no objections for about a week, I've pushed the 0001
> patch. Thanks!
>=20
> If any issues or concerns are raised, let's consider applying the 0002 =
patch!
>=20
> Regards,
>=20
> --=20
> Fujii Masao

TBH, I=E2=80=99m not quite sure what practical benefit this patch set =
provides.

In 0001, some messages are changed from LOG to DEBUG1. But then 0002 =
introduces trace_logical_decoding_messages, whose default is DEBUG1, so =
with the default setting those messages are effectively promoted back to =
LOG again. That seems to cancel out the behavior change from 0001 by =
default.

Also, looking at where trace_logical_decoding() is applied in 0002, all =
the call sites currently pass DEBUG1. So in practice, if a user sets =
trace_logical_decoding_messages to anything higher than DEBUG1, all of =
those messages disappear. That makes it feel more like an on/off switch =
than a mechanism for filtering messages by level.

So at least to me, the overall value of this patch is still not very =
clear.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/