Re: [HACKERS] proposal psql \gdesc

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: [HACKERS] proposal psql \gdesc
Дата
Msg-id CAFj8pRDHs9DnoXEKOYdkGV2-YEPiq-wj7_Q+XJsZrTUQMgegWQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [HACKERS] proposal psql \gdesc  (Fabien COELHO <coelho@cri.ensmp.fr>)
Ответы Re: [HACKERS] proposal psql \gdesc  (Fabien COELHO <coelho@cri.ensmp.fr>)
Список pgsql-hackers


2017-05-09 20:37 GMT+02:00 Fabien COELHO <coelho@cri.ensmp.fr>:

Hello Pavel,

[...] it is little bit worse. I cannot to distinguish between SELECT\gdesc and TRUNCATE xxx\gdesc . All are valid commands and produce empty result, so result of \gdesc command should be empty result too.

postgres=# truncate table xx\gdesc
┌──────┬──────┐
│ Name │ Type │
╞══════╪══════╡
└──────┴──────┘
(0 rows)

Hmmm. At least it is better than the previous error.

What about detecting the empty result (eg PQntuples()==0?) and writing "Empty result" instead of the strange looking empty table above? That would just mean skipping the PrintQueryResult call in this case?

PQntuples == 0 every time - the query is not executed.

I am not sure, what is more correct. The "Empty result" string is not used every time in psql. For the case "SELECT;" the empty table is correct. For TRUNCATE and similar command I am not sure. The empty table is maybe unusual, but it is valid - like "SELECT;". The implementation is not problem in any case. The question is what is more natural for users a) the string "Empty result", b) empty table. 

I prefer @b due consistency with current behave (that is only reason, I have not any other).

postgres=# select 'ahoj' where false;
┌──────────┐
│ ?column? │
╞══════════╡
└──────────┘
(0 rows)

 


--
Fabien.

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Petr Jelinek
Дата:
Сообщение: Re: [HACKERS] snapbuild woes
Следующее
От: Peter Eisentraut
Дата:
Сообщение: Re: [HACKERS] CTE inlining