Re: log_hostname and pg_stat_activity
Steve Singer <ssinger_pg@sympatico.ca>
From: Steve Singer <ssinger_pg@sympatico.ca>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Peter Eisentraut <peter_e@gmx.net>, pgsql-hackers@postgresql.org
Date: 2011-02-10T15:22:53Z
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@gmail.com> wrote: >> On Tue, Feb 1, 2011 at 1:09 PM, Peter Eisentraut<peter_e@gmx.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.