Re: Re: [PATCH] Add support for ON UPDATE/DELETE actions on ALTERCONSTRAINT - Mailing list pgsql-hackers

From David Steele
Subject Re: Re: [PATCH] Add support for ON UPDATE/DELETE actions on ALTERCONSTRAINT
Date
Msg-id 61a6f06c-2b27-444a-ca0d-11d053839e27@pgmasters.net
Whole thread Raw
In response to Re: [PATCH] Add support for ON UPDATE/DELETE actions on ALTERCONSTRAINT  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
Hi Matheus,

On 3/3/18 1:32 PM, Peter Eisentraut wrote:
> On 2/20/18 10:10, Matheus de Oliveira wrote:
>> Besides that, there is a another change in this patch on current ALTER
>> CONSTRAINT about deferrability options. Previously, if the user did
>> ALTER CONSTRAINT without specifying an option on deferrable or
>> initdeferred, it was implied the default options, so this:
>>
>>     ALTER TABLE tbl
>>     ALTER CONSTRAINT con_name;
>>
>> Was equivalent to:
>>
>>     ALTER TABLE tbl
>>     ALTER CONSTRAINT con_name NOT DEFERRABLE INITIALLY IMMEDIATE;
> 
> Oh, that seems wrong.  Probably, it shouldn't even accept that syntax
> with an empty options list, let alone reset options that are not
> mentioned.  Can you prepare a separate patch for this issue?

Can you prepare the patch that Peter has requested and post on a new
thread?  Please respond here with the reference (or email me directly)
and I will add to the CF.

Meanwhile, I'll push this patch to the next CF as Andres has
recommended, hearing no arguments to the contrary.

Thanks,
-- 
-David
david@pgmasters.net


pgsql-hackers by date:

Previous
From: David Steele
Date:
Subject: Re: Re: Boolean partitions syntax
Next
From: David Steele
Date:
Subject: Re: Re: [HACKERS] Can ICU be used for a database's default sortorder?