Re: ALTER TABLE ... ALTER CONSTRAINT

From: Jeff Janes <jeff(dot)janes(at)gmail(dot)com>
To: Abhijit Menon-Sen <ams(at)2ndquadrant(dot)com>
Cc: Simon Riggs <simon(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: ALTER TABLE ... ALTER CONSTRAINT
Date: 2013-06-24 20:42:45
Message-ID: CAMkU=1xjGqkeC4gmp_bz0YBoXznkzBvV0WUA18ZqqXRzM14big@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Jun 23, 2013 at 8:58 PM, Abhijit Menon-Sen <ams(at)2ndquadrant(dot)com>wrote:

> At 2013-06-08 21:45:24 +0100, simon(at)2ndQuadrant(dot)com wrote:
> >
> > ALTER TABLE foo
> > ALTER CONSTRAINT fktable_fk_fkey DEFERRED INITIALLY IMMEDIATE;
>
> I read the patch (looks good), applied it on HEAD (fine), ran make check
> (fine), and played with it in a test database. It looks great, and from
> previous responses it's something a lot of people have wished for.
>
> I'm marking this ready for committer.
>

After the commit, I'm now getting the compiler warning:

tablecmds.c: In function 'ATPrepCmd':
tablecmds.c:2953: warning: 'pass' may be used uninitialized in this function

case AT_AlterConstraint (line 3130) is the only case branch that does not
set pass.

Cheers,

Jeff

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2013-06-24 20:50:58 Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review]
Previous Message Josh Berkus 2013-06-24 20:32:41 Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review]