Re: log_hostname and pg_stat_activity

From: Steve Singer <ssinger_pg(at)sympatico(dot)ca>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: log_hostname and pg_stat_activity
Date: 2011-02-10 15:22:53
Message-ID: BLU0-SMTP6B1C3E95E9682ADF432B18EEC0@phx.gbl
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 11-02-10 10:13 AM, Robert Haas wrote:
> On Tue, Feb 1, 2011 at 1:33 PM, Robert Haas<robertmhaas(at)gmail(dot)com> wrote:
>> On Tue, Feb 1, 2011 at 1:09 PM, Peter Eisentraut<peter_e(at)gmx(dot)net> wrote:
>>> On tis, 2011-01-18 at 19:24 -0500, Steve Singer wrote:
>>>> However if I connect with a line in pg_hba that matches on an IP
>>>> network then my client_hostname is always null unless log_hostname is
>>>> set to true. This is consistent with the behavior you describe but I
>>>> think the average user will find it a bit confusing. Having a column
>>>> that is always null unless a GUC is set is less than ideal but I
>>>> understand why log_hostname isn't on by default.
>>> Well, we have all these track_* variables, which also control what
>>> appears in the statistics views.
>>>
>>> After thinking about this some more, I think it might be better to be
>>> less cute and forget about the interaction with the pg_hba.conf hostname
>>> behavior. That is, the host name is set if and only if log_hostname is
>>> on.
>> +1 for doing it that way.
> I think there are no outstanding issues with this patch of any
> significance, so I'm marking it Ready for Committer.
>
Was there an uodated version of this patch I missed?

The original patch needed some sort of documentation saying that having
something showup in the new pg_stat_activity columns is controlled by
log_hostname.

Above Peter and you seem to agree that having the having the line
matched in pg_hba being a controlling factor should be removed but I
haven't seen an updated patch that implements that.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Θάνος Παπαπέτρου 2011-02-10 15:23:12 Adding new variables into GUC
Previous Message Robert Haas 2011-02-10 15:22:31 Re: pg_dump directory archive format / parallel pg_dump