Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Simon Riggs
On Wed, Jun 15, 2011 at 10:42 PM, Tom Lane wrote: > Alvaro Herrera writes: >> So is somebody from 2nd Quadrant going to supply a patch to fix this? > > I'm already on it.  The whole patch appears to need some review, > considering this is about the fourth major flaw we've found in it. I'll leave

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Simon Riggs
On Wed, Jun 15, 2011 at 9:59 PM, Alvaro Herrera wrote: > Excerpts from Simon Riggs's message of mié jun 15 16:31:45 -0400 2011: >> On Wed, Jun 15, 2011 at 9:14 PM, Alvaro Herrera >> wrote: >> >> > So is somebody from 2nd Quadrant going to supply a patch to fix this? >> >> My understanding was tha

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Tom Lane
Alvaro Herrera writes: > So is somebody from 2nd Quadrant going to supply a patch to fix this? I'm already on it. The whole patch appears to need some review, considering this is about the fourth major flaw we've found in it. regards, tom lane -- Sent via pgsql-hackers

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Alvaro Herrera
Excerpts from Simon Riggs's message of mié jun 15 16:31:45 -0400 2011: > On Wed, Jun 15, 2011 at 9:14 PM, Alvaro Herrera > wrote: > > > So is somebody from 2nd Quadrant going to supply a patch to fix this? > > My understanding was that your patch had a bug, rather than the > existing code. If I

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Simon Riggs
On Wed, Jun 15, 2011 at 9:14 PM, Alvaro Herrera wrote: > So is somebody from 2nd Quadrant going to supply a patch to fix this? My understanding was that your patch had a bug, rather than the existing code. If I misunderstood, please explain the bug. In terms of "2ndQuadrant" supplying patches,

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Jaime Casanova
On Wed, Jun 15, 2011 at 3:14 PM, Alvaro Herrera wrote: > > So is somebody from 2nd Quadrant going to supply a patch to fix this? > well, i was going to give it a try... but in a couple of hours... -- Jaime Casanova         www.2ndQuadrant.com Professional PostgreSQL: Soporte 24x7 y capacitación

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Alvaro Herrera
Excerpts from Tom Lane's message of mié jun 15 11:54:25 -0400 2011: > Dean Rasheed writes: > > On 15 June 2011 07:56, Jaime Casanova wrote: > >> Testing the CHECK NOT VALID patch i found $subject... is this intended? > > > Aside from the ugliness of the code, we can't just add a > > ConstraintAt

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Simon Riggs
On Wed, Jun 15, 2011 at 4:54 PM, Tom Lane wrote: > Dean Rasheed writes: >> On 15 June 2011 07:56, Jaime Casanova wrote: >>> Testing the CHECK NOT VALID patch i found $subject... is this intended? > >> Aside from the ugliness of the code, we can't just add a >> ConstraintAttributeSpec to the seco

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Tom Lane
Dean Rasheed writes: > On 15 June 2011 07:56, Jaime Casanova wrote: >> Testing the CHECK NOT VALID patch i found $subject... is this intended? > Aside from the ugliness of the code, we can't just add a > ConstraintAttributeSpec to the second block, because that would > enforce an order to these

Re: [HACKERS] FK NOT VALID can't be deferrable?

2011-06-15 Thread Dean Rasheed
On 15 June 2011 07:56, Jaime Casanova wrote: > Hi, > > Testing the CHECK NOT VALID patch i found $subject... is this intended? > I just noticed that too, and was about to raise it as a bug. If it is intended, then it's not documented. I noticed it while browsing gram.y, and thought it looks a b

[HACKERS] FK NOT VALID can't be deferrable?

2011-06-14 Thread Jaime Casanova
Hi, Testing the CHECK NOT VALID patch i found $subject... is this intended? -- Jaime Casanova         www.2ndQuadrant.com Professional PostgreSQL: Soporte 24x7 y capacitación -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.