Re: [PATCH] Connection time for \conninfo

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [PATCH] Connection time for \conninfo
Дата
Msg-id 8078.1583863400@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [PATCH] Connection time for \conninfo  (Stephen Frost <sfrost@snowman.net>)
Ответы Re: [PATCH] Connection time for \conninfo  (Stephen Frost <sfrost@snowman.net>)
Список 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 по дате отправления:

Предыдущее
От: Peter Eisentraut
Дата:
Сообщение: Re: [PATCH] Connection time for \conninfo
Следующее
От: Stephen Frost
Дата:
Сообщение: Re: [PATCH] Connection time for \conninfo