Postgres Pro
Downloads
Home   >   mailing lists

Re: [HACKERS] Re: proposal - psql: possibility to specify sort fordescribe commands, when size is printed - Mailing list pgsql-hackers

From Alexander Korotkov
Subject Re: [HACKERS] Re: proposal - psql: possibility to specify sort fordescribe commands, when size is printed
Date
Msg-id CAPpHfdsLq51crn2ome7y5TMEh-HQaXVaXeQ-xO2Xt9h8MQn7NQ@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] Re: proposal - psql: possibility to specify sort fordescribe commands, when size is printed  (Pavel Stehule <pavel.stehule@gmail.com>)
Responses Re: [HACKERS] Re: proposal - psql: possibility to specify sort fordescribe commands, when size is printed  (Pavel Stehule <pavel.stehule@gmail.com>)
List pgsql-hackers
Tree view
On Fri, Sep 8, 2017 at 7:13 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
2017-08-16 14:06 GMT+02:00 Pavel Stehule <pavel.stehule@gmail.com>:
Hi

2017-08-15 4:37 GMT+02:00 Peter Eisentraut <peter.eisentraut@2ndquadrant.com>:
On 3/11/17 07:06, Pavel Stehule wrote:
> I am sending a updated version with separated sort direction in special
> variable

This patch also needs a rebase.

I am sending rebased patch

rebased again + fix obsolete help

For me, patch applies cleanly, builds and passed regression tests.
However, patch misses regression tests covering added functionality.
Patch is definitely harmless, i.e. it doesn't affect anybody who doesn't use new functionality.
But I still would prefer ordering to be options of \d* commands while psql variables be defaults for those options...

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company 

pgsql-hackers by date:

Previous
From: Robins Tharakan
Date:
Subject: Re: [HACKERS] psql - add ability to test whether a variable exists
Next
From: Andrew Dunstan
Date:
Subject: [HACKERS] Re: [COMMITTERS] pgsql: Add citext_pattern_ops for citext contribmodule