From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
---|---|
To: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: proposal psql \gdesc |
Date: | 2017-05-20 07:15:39 |
Message-ID: | alpine.DEB.2.20.1705200845450.6053@lancre |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello Pavel,
> I am sending a variant with message instead empty result.
Thanks. Patch looks good, applies, make check ok, code is neat.
> Personally I prefer empty result instead message.
Hmm. I think that this version is less likely to raise questions from
users, especially compared to having a somehow correct but strangely
looking description.
> It is hard to choose some good text of this message. Empty result is
> just empty result for all cases.
I'd suggest a very minor change: "No columns or command has no result"
(not -> no). If some English native speaker has a better suggestion,
fine with me.
Another good point of this version is that the type name query is
simplified because it does not need to handle an empty result, thus the
code is easier to understand.
A few other suggestions:
- could you update the comment on the type name query?
Maybe the comment can be simply removed?
- I'm wondering whether the Name & Type columns names should be
translatable. What do you think?
- Maybe tests could also exercise unnamed columns, eg:
SELECT 1, 2, 3 \gdesc \g
--
Fabien.
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2017-05-20 07:56:57 | Re: Removal of plaintext password type references |
Previous Message | Pavel Stehule | 2017-05-20 06:25:32 | Re: Re: proposal - using names as primary names of plpgsql function parameters instead $ based names |