Hi 2017-09-19 16:14 GMT+02:00 Alexander Korotkov <a.korot...@postgrespro.ru>:
> On Fri, Sep 8, 2017 at 7:13 AM, Pavel Stehule <pavel.steh...@gmail.com> > wrote: > >> 2017-08-16 14:06 GMT+02:00 Pavel Stehule <pavel.steh...@gmail.com>: >> >>> Hi >>> >>> 2017-08-15 4:37 GMT+02:00 Peter Eisentraut < >>> peter.eisentr...@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. > I am not sure if there are any tests related to output of \dt+ commands - there result is platform depend. > 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... > I understand a) I don't think so commands like \dt++ (or similar) is good idea - these commands should be simple how it is possible b) this patch doesn't block any other design - more it opens the door because the executive part will be implemented and users can have a experience with with different output sorts - so if people will need more quick change of result sort, then the work in this area will continue. Regards Pavel > ------ > Alexander Korotkov > Postgres Professional: http://www.postgrespro.com > The Russian Postgres Company >
