2009/7/14 Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec>:
> On Thu, Mar 26, 2009 at 2:39 AM, higepon<higepon(at)gmail(dot)com> wrote:
>> Here is a patch for pg_dump "Commenting on a composite-type column".
>> This patch is for Todo item named "Add dumping of comments on index
>> columns and composite type columns".
> this one looks good to me, the only adjust i made to the patch is
> change the name for the function that dump the comments from the
> composite types columns for: dumpCompositeTypeColsComment that seems
> more clearer to me...
> the patch works just fine...
Oops. I picked this patch up from the commitfest queue because it was
still marked as "Needs Review", not realising that Jaime had already
done an initial review of the patch.
Seems like this one should have been "Ready for Committer", but no matter.
I've also done an initial review of the patch. Everything looks sane
and the patch works as advertised. I made a couple of minor tweaks
for code-style and comment consistency, and my version 3 is attached.
I'm marking this patch Ready for Committer.
I've also added a TODO item "forbid COMMENT on columns of an index",
per Tom's comments upthread.
In response to
pgsql-hackers by date
|Next:||From: Magnus Hagander||Date: 2009-07-22 06:52:25|
|Subject: Re: CommitFest 2009-07 - End of Week 1|
|Previous:||From: Tom Lane||Date: 2009-07-22 03:35:30|
|Subject: Re: CommitFest 2009-07 - End of Week 1 |