Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2017-01-25 Thread Dave Page
Thanks, patch applied. On Fri, Jan 20, 2017 at 7:29 AM, Surinder Kumar wrote: > Hi Dave, > > The last patch was for exclusion constraint only. The same issue issue is > reported RM2061 for index. > So, this patch takes care of both issues. > > Added a flag

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2017-01-19 Thread Surinder Kumar
Hi Dave, The last patch was for exclusion constraint only. The same issue issue is reported RM2061 for index. So, this patch takes care of both issues. Added a flag '*is_sort_nulls_applicable*' which is set to true for access method 'btree'. If its is

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2017-01-17 Thread Dave Page
Hi On Tue, Jan 17, 2017 at 9:52 AM, Surinder Kumar wrote: > Hi > On Tue, Jan 17, 2017 at 3:08 PM, Dave Page wrote: >> >> Hi >> >> On Fri, Jan 13, 2017 at 6:50 AM, Surinder Kumar >> wrote: >> > Hi >> > >> >

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2017-01-17 Thread Dave Page
Hi On Fri, Jan 13, 2017 at 6:50 AM, Surinder Kumar wrote: > Hi > > Please find updated patch following changes: > 1) Keep field 'opclass' combo box enabled. That doesn't seem to be working. > 2) Keep ASC/DESC and NULLs FIRST/LAST options disable for access

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2017-01-12 Thread Surinder Kumar
Hi Please find updated patch following changes: 1) Keep field 'opclass' combo box enabled. 2) Keep ASC/DESC and NULLs FIRST/LAST options disable for access methods other than 'btree'. 3) Add validation for name field. On Fri, Nov 25, 2016 at 5:16 PM, Dave Page wrote: > On

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2016-11-25 Thread Dave Page
On Thu, Nov 24, 2016 at 12:13 PM, Surinder Kumar wrote: > Hi > > Please find updated patch with change. > > On Fri, Oct 21, 2016 at 9:16 PM, Dave Page wrote: >> >> On Fri, Oct 21, 2016 at 4:42 PM, Surinder Kumar >>

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2016-11-24 Thread Surinder Kumar
Hi Please find updated patch with change. On Fri, Oct 21, 2016 at 9:16 PM, Dave Page wrote: > On Fri, Oct 21, 2016 at 4:42 PM, Surinder Kumar > wrote: > > On Fri, Oct 21, 2016 at 8:52 PM, Dave Page wrote: > >> > >> Hi >

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2016-10-21 Thread Dave Page
On Fri, Oct 21, 2016 at 4:42 PM, Surinder Kumar wrote: > On Fri, Oct 21, 2016 at 8:52 PM, Dave Page wrote: >> >> Hi >> >> On Fri, Oct 21, 2016 at 4:16 PM, Surinder Kumar >> wrote: >> > Hi >> > >> > This fix is

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2016-10-21 Thread Surinder Kumar
On Fri, Oct 21, 2016 at 8:52 PM, Dave Page wrote: > Hi > > On Fri, Oct 21, 2016 at 4:16 PM, Surinder Kumar > wrote: > > Hi > > > > This fix is for exclusion constraint. > > The options like "order" and "nulls" must be conditional. i.e. include

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2016-10-21 Thread Dave Page
Hi On Fri, Oct 21, 2016 at 4:16 PM, Surinder Kumar wrote: > Hi > > This fix is for exclusion constraint. > The options like "order" and "nulls" must be conditional. i.e. include only > when access method type is other than "gist". When creating an index, the

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2016-10-21 Thread Surinder Kumar
Hi This fix is for exclusion constraint. The options like "order" and "nulls" must be conditional. i.e. include only when access method type is other than "gist". Please find attached patch and review. On Fri, Oct 21, 2016 at 4:38 PM, Dave Page wrote: > Thanks, applied. > >

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1840 - cannot create gist index due to enforced ASC, DESC options in generated SQL

2016-10-21 Thread Dave Page
Thanks, applied. On Thu, Oct 20, 2016 at 7:47 AM, Surinder Kumar wrote: > Hi, > > The options like "sort" and "nulls" must be conditional. i.e. include only > when access method type is other than "gist" or "gin". > > Please find attached patch and review. > >