On 2020-Nov-23, Tom Lane wrote: > Ah, I see I didn't cover the case in ProcSleep that you were originally on > about ... I'd just looked for existing references to log_min_messages > and client_min_messages.
Yeah, it seemed bad form to add that when you had just argued against it :-) > I think it's important to have the explicit check for elevel >= ERROR. > I'm not too fussed about whether we invent is_log_level_output_client, > although that name doesn't seem well-chosen compared to > is_log_level_output. Just replacing "log" for "client" in that seemed strictly worse, and I didn't (don't) have any other ideas. > Shall I press forward with this, or do you want to? Please feel free to go ahead, including the change to ProcSleep.