Re: [PATCH] Connection time for \conninfo - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [PATCH] Connection time for \conninfo
Date
Msg-id 20190904150403.GA27872@alvherre.pgsql
Whole thread Raw
In response to Re: [PATCH] Connection time for \conninfo  (Michael Paquier <michael@paquier.xyz>)
Responses Re: [PATCH] Connection time for \conninfo
List pgsql-hackers
On 2019-Sep-04, Michael Paquier wrote:

> On Tue, Sep 03, 2019 at 10:13:57PM -0400, Rodrigo Ramírez Norambuena wrote:
> > I've work in the a little patch to add into the \conninfo of psql
> > command the connection time  against a server backend
> > 
> > Maybe could add after, the precheck to if the connection is alive.
> > I've take first look to the pqPacketSend, my idea is send a ECHO
> > packet over to the database connection. If someone has a better idea
> > it would be great to know.
> 
> You can do basically the same thing by looking at
> pg_stat_activity.backend_start and compare it with the clock
> timestamp.  Doing it at SQL level the way you want has also the
> advantage to offer you a modular format output.

Hmm, couldn't you say the same about the other details that \conninfo
gives?  You can get them from pg_stat_activity or pg_stat_ssl, yet
they're still shown \conninfo for convenience.  Surely we don't want to
remove everything from \conninfo and tell users to query the stat views
instead.

The only thing that seems wrong about this proposal is that the time
format is a bit too verbose.  I would have it do "N days 12:23:34".

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Unexpected "shared memory block is still in use"
Next
From: Alvaro Herrera
Date:
Subject: auxiliary processes in pg_stat_ssl