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

From Peter Eisentraut
Subject Re: [PATCH] Add support for ON UPDATE/DELETE actions on ALTERCONSTRAINT
Date
Msg-id 962fa8bb-6268-e958-c702-095ff6bf81aa@2ndquadrant.com
Whole thread Raw
In response to [PATCH] Add support for ON UPDATE/DELETE actions on ALTER CONSTRAINT  (Matheus de Oliveira <matioli.matheus@gmail.com>)
Responses Re: Re: [PATCH] Add support for ON UPDATE/DELETE actions on ALTERCONSTRAINT  (David Steele <david@pgmasters.net>)
Re: [PATCH] Add support for ON UPDATE/DELETE actions on ALTER CONSTRAINT  (Matheus de Oliveira <matioli.matheus@gmail.com>)
List pgsql-hackers
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?

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: Online enabling of checksums
Next
From: Justin Pryzby
Date:
Subject: Re: Function to track shmem reinit time