| From: | Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> |
|---|---|
| To: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
| Cc: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Reduce log level of some logical decoding messages to DEBUG1 |
| Date: | 2026-04-01 07:30:40 |
| Message-ID: | 5AFD043F-6542-4BE8-BC46-5F9EF281B3BF@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On Apr 1, 2026, at 14:48, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
>
> On Wed, Mar 25, 2026 at 2:54 PM Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
>>> Perhaps, we could wait
>>> for some time to hear from others.
>>
>> +1.
>>
>> 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).
>
> Since there were no objections for about a week, I've pushed the 0001
> patch. Thanks!
>
> If any issues or concerns are raised, let's consider applying the 0002 patch!
>
> Regards,
>
> --
> Fujii Masao
TBH, I’m 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/
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniil Davydov | 2026-04-01 07:44:25 | Re: POC: Parallel processing of indexes in autovacuum |
| Previous Message | jian he | 2026-04-01 07:23:07 | Re: CAST(... ON DEFAULT) - WIP build on top of Error-Safe User Functions |