Re: [PATCH] Include application_name in "connection authorized" log message - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [PATCH] Include application_name in "connection authorized" log message
Date
Msg-id CA+TgmoaeHYvbOsHKFZJONzbgodU7deR=E+sAm=pooHTMRshn-g@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Include application_name in "connection authorized" logmessage  (Stephen Frost <sfrost@snowman.net>)
Responses Re: [PATCH] Include application_name in "connection authorized" logmessage
List pgsql-hackers
On Wed, Jun 20, 2018 at 3:45 PM, Stephen Frost <sfrost@snowman.net> wrote:
> * Don Seiler (don@seiler.us) wrote:
>> In trying to troubleshoot the source of a recent connection storm, I was
>> frustrated to find that the app name was not included in the connection
>> messages. It is there in the log_line_prefix on the disconnection messages
>> but I would prefer it be directly visible with the connection itself. With
>> some guidance from Stephen Frost I've put together this patch which does
>> that.
>
> Yeah, I tend to agree that it'd be extremely useful to have this
> included in the 'connection authorized' message.

I don't get it.  It seems like a bad idea to me to copy information
that can already be logged using log_line_prefix into the message
itself.  If we start doing that, we'll end up with duplicated
information all over the place, and even worse, it won't be
consistent, because different people will want different things
duplicated into different messages.

Am I missing something?

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Expression errors with "FOR UPDATE" and postgres_fdw withpartition wise join enabled.
Next
From: Robert Haas
Date:
Subject: Re: New function pg_stat_statements_reset_query() to reset statisticsof a specific query