Patch applied.  Thanks.

I ran pgindent on the tablecmds.c block of code, and cleaned up some
boolean assignments.  There are a few XXX comments still in the code so
someone should look at those questions and either modify the code or
remove the comments.


Greg Stark wrote:
> I cleaned up the code and added some more documentation.
> I think I've addressed all the concerns raised so far. Please tell me if I've
> missed anything.
> There were a few tangentially related issues that have come up that I think
> are TODOs. I'm likely to tackle one or two of these next so I'm interested in
> hearing feedback on them as well.
> . Constraints currently do not know anything about inheritance. Tom suggested
>   adding a coninhcount and conislocal like attributes have to track their
>   inheritance status.
> . Foreign key constraints currently do not get copied to new children (and
>   therefore my code doesn't verify them). I don't think it would be hard to
>   add them and treat them like CHECK constraints.
> . No constraints at all are copied to tables defined with LIKE. That makes it
>   hard to use LIKE to define new partitions. The standard defines LIKE and
>   specifically says it does not copy constraints. But the standard already has
>   an option called INCLUDING DEFAULTS; we could always define a non-standard
>   extension LIKE table INCLUDING CONSTRAINTS that gives the user the option to
>   request a copy including constraints.
> . Personally, I think the whole attislocal thing is bunk. The decision about
>   whether to drop a column from children tables or not is something that
>   should be up to the user and trying to DWIM based on whether there was ever
>   a local definition or the column was acquired purely through inheritance is
>   hardly ever going to match up with user expectations.
> . And of course there's the whole unique and primary key constraint issue. I
>   think to get any traction at all on this you have a prerequisite of a real
>   partitioned table implementation where the system knows what the partition
>   key is so it can recognize when it's a leading part of an index key. 

[ Attachment, skipping... ]

> -- 
> greg

> ---------------------------(end of broadcast)---------------------------
> TIP 2: Don't 'kill -9' the postmaster

  Bruce Momjian   [EMAIL PROTECTED]

  + If your life is a hard drive, Christ can be your backup. +

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?


Reply via email to