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

From Tom Lane
Subject Re: [PATCH] Connection time for \conninfo
Date
Msg-id 8078.1583863400@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] Connection time for \conninfo  (Stephen Frost <sfrost@snowman.net>)
Responses Re: [PATCH] Connection time for \conninfo  (Stephen Frost <sfrost@snowman.net>)
List pgsql-hackers
Stephen Frost <sfrost@snowman.net> writes:
> Anyway, I don't anticipate having time to do anything with this patch
> but I disagree that this is a "we don't want it" kind of thing, rather
> we maybe want it, since someone cared enough to write a patch, but the
> patch needs work and maybe we want it to look a bit different and be
> better defined.

I think Peter's primary argument was that this doesn't belong in
\conninfo, which is about reporting the parameters required to
establish the connection.  We have kind of broken that already by
cramming SSL and GSS encryption info into the results, but that
doesn't mean it should become a kitchen-sink listing of anything
anybody says they'd like to know.

Anyway, I think your point is that maybe this should be RWF
not Rejected, and I agree with that.

(I had not looked at the last version of the patch, but now that
I have, I still don't like the fact that it has the client tracking
session start time separately from what the server does.  The small
discrepancy that introduces is going to confuse somebody.  I see
that there's no documentation update either.)

            regards, tom lane



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: [PATCH] Connection time for \conninfo
Next
From: Stephen Frost
Date:
Subject: Re: [PATCH] Connection time for \conninfo