Re: [HACKERS] review patch: Distinguish between unique indexes and unique constraints

2010-08-01 Thread Robert Haas
On Sun, Aug 1, 2010 at 6:54 AM, Kevin Grittner wrote: > Robert Haas  wrote: >> I have committed this patch with a few changes. > > Thanks. > >> First, I felt that there was little point in showing this detail >> only in verbose mode; indeed, it seems like that could be confusing >> in some circums

Re: [HACKERS] review patch: Distinguish between unique indexes and unique constraints

2010-08-01 Thread Kevin Grittner
Robert Haas wrote: > I have committed this patch with a few changes. Thanks. > First, I felt that there was little point in showing this detail > only in verbose mode; indeed, it seems like that could be confusing > in some circumstances. (I thought I checked this was an index not > a const

Re: [HACKERS] review patch: Distinguish between unique indexes and unique constraints

2010-07-31 Thread Robert Haas
On Thu, Jul 29, 2010 at 7:56 PM, Kevin Grittner wrote: > The patch is in context diff format and applies cleanly.  No doc > changes were included.  Arguably there should be a mention in the > documentation for psql's \d+ commend, but since the number of child > tables and the display of reloptions

[HACKERS] review patch: Distinguish between unique indexes and unique constraints

2010-07-29 Thread Kevin Grittner
The patch is in context diff format and applies cleanly. No doc changes were included. Arguably there should be a mention in the documentation for psql's \d+ commend, but since the number of child tables and the display of reloptions aren't mentioned, perhaps we're not trying to list *all* the di