Skip site navigation (1) Skip section navigation (2)

Re: the patch: support for DESC/NULLS FIRST/NULLS LAST

From: Quan Zongliang <quanzongliang(at)gmail(dot)com>
To: "Dave Page" <dpage(at)pgadmin(dot)org>
Cc: "Guillaume Lelarge" <guillaume(at)lelarge(dot)info>, pgadmin-hackers(at)postgresql(dot)org
Subject: Re: the patch: support for DESC/NULLS FIRST/NULLS LAST
Date: 2008-11-17 10:00:39
Message-ID: 20081117182745.C8CA.4125B4E5@gmail.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
> The case of the text should follow existing cases as well - if a label
In fact, NULLS order's default value is different base of DESC.
When create index:
If DESC is specified, default is NULLS FIRST.
If not (ASC, default), default is NULLS LAST.

So, I think if there is only FIRST and LAST. Maybe it confuse user.

Suggest me, pls.

> The DESC checkbox doesn't align horizontally with the combo box to its left.

Like this: ?
Column    <column's combobox>
DESC      <checkbox>
NULLS     <Default   FIRST   LAST>
                      Add  Remove

> The columns list view doesn't size correctly when the form is opened
> on Mac
Yes, new method dlgIndex::OnChangeSize() is needed.
I have not Mac, can't test.

PS:
About TODO item:
Allow Sequence to be attached to existing int4/int8 columns

It means to make sequence dialog can support the "OWNED BY" clause in
"CREATE/ALTER SEQUENCE" statement?


-----------------------------------------------
Quan Zongliang
quanzongliang(at)gmail(dot)com
CIT Japan:  http://www.cit.co.jp


In response to

Responses

pgadmin-hackers by date

Next:From: Dave PageDate: 2008-11-17 10:25:08
Subject: Re: the patch: support for DESC/NULLS FIRST/NULLS LAST
Previous:From: Dave PageDate: 2008-11-17 09:03:23
Subject: Re: the patch: support for DESC/NULLS FIRST/NULLS LAST

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group