On Wed, Jan 26, 2022 at 9:48 AM Fujii Masao <masao.fu...@oss.nttdata.com> wrote:
>
> Hi,
>
> pg_log_backend_memory_contexts() should be designed not to send the messages 
> about the memory contexts to the client regardless of client_min_messages. 
> But I found that the message "logging memory contexts of PID %d" can be sent 
> to the client because it's ereport()'d with LOG level instead of 
> LOG_SERVER_ONLY. Is this a bug, and shouldn't we use LOG_SERVER_ONLY level to 
> log that message? Patch attached.

+1. The patch LGTM.

The same applies for the on-going patch [1] for pg_log_backtrace, let
me quickly send the updated patch there as well. The pg_log_query_plan
on-going patch [2], does the right thing.

[1] - 
https://www.postgresql.org/message-id/CALDaNm3tbc1OKvSKvD5SfmEj66M_sWDPCgDvzFtS9gxRov8jRQ%40mail.gmail.com
[2] - 
https://www.postgresql.org/message-id/fb6360e4c0ffbdd0a6698b92bb2617e2%40oss.nttdata.com

Regards,
Bharath Rupireddy.


Reply via email to