Re: [patch] Proposal for \rotate in psql

Поиск
Список
Период
Сортировка
От Daniel Verite
Тема Re: [patch] Proposal for \rotate in psql
Дата
Msg-id e3966df0-6716-4899-8d62-c0ea95c3debb@mm
обсуждение исходный текст
Ответ на Re: [patch] Proposal for \rotate in psql  (Pavel Stehule <pavel.stehule@gmail.com>)
Ответы Re: [patch] Proposal for \rotate in psql  (Pavel Stehule <pavel.stehule@gmail.com>)
Список pgsql-hackers
    Pavel Stehule wrote:


> my comments:
>
> 1. I don't understand why you are use two methods for sorting columns
> (qsort, and query with ORDER BY)

qsort (with strcmp as the comparator) is only used to determine the
set of distinct values for the vertical and horizontal headers.
In fact this is just to allow the use of bsearch() when doing that
instead of a slower sequential search.

Once the values for the horizontal headers are known, they
are passed to the server for sorting with server-side semantics
according to their type. The order will differ from qsort/strcmp
found as the comparison semantics are different.

The values for the vertical header are not sorted server-side
because we keep the order of the query for displaying
top to bottom.
In the typical use case , it will have ORDER BY 1[,2] possibly
with one/two DESC qualifiers.

> 2. Data column are not well aligned - numbers are aligned as text

Yes. I'll look shortly into fixing that.


Best regards,
--
Daniel Vérité
PostgreSQL-powered mailer: http://www.manitou-mail.org
Twitter: @DanielVerite



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

Предыдущее
От: Pavel Stehule
Дата:
Сообщение: Re: [patch] Proposal for \rotate in psql
Следующее
От: Robert Haas
Дата:
Сообщение: Re: On-demand running query plans using auto_explain and signals