From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Why does logical replication launcher set application_name? |
Date: | 2017-04-12 00:32:20 |
Message-ID: | CA+TgmoahvHugMoqwMUe+-tABETEutCh6OV7gO7g_dcemSC6cbw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Apr 11, 2017 at 8:11 PM, Michael Paquier
<michael(dot)paquier(at)gmail(dot)com> wrote:
> On Wed, Apr 12, 2017 at 3:40 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> I notice looking at pg_stat_activity that the logical replication launcher
>> sets its application_name to "logical replication launcher". This seems
>> inconsistent (no other standard background process sets application_name),
>> redundant with other columns that already tell you what it is, and an
>> unreasonable consumption of horizontal space in the tabular output.
>> Can we drop that? If we do have to have something like that, what about
>> putting it in the "query" field where it's much less likely to be
>> substantially wider than any other entry in the column?
>
> It seems to me that the logic behind that is to be able to identify
> easily the logical replication launcher in pg_stat_activity, so using
> the query field instead sounds fine for me as we need another way than
> backend_type to guess what is this bgworker.
Wait, why do we need two ways?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2017-04-12 00:41:18 | Re: Some thoughts about SCRAM implementation |
Previous Message | Michael Paquier | 2017-04-12 00:22:47 | Re: TAP tests take a long time |