Re: gSoC - ADD MERGE COMMAND - code patch submission

David Fetter <david@fetter.org> writes:
> On Sat, Jul 10, 2010 at 01:53:53PM -0400, Tom Lane wrote:
>> BTW, this patch is still a few bricks shy of a load, since there's
>> no kwlist.h change and so the new MERGE keyword couldn't possibly be
>> recognized.  More generally, I'm wondering why the original .rar
>> submission was 300k (presumably compressed) and your diff is only
>> about 35k ...

> I'll look into that.  From what you can see, is it worth trying to
> clean up, starting from base, or should we just wait for the next
> revision of the patch?

Well, rebasing against HEAD will presumably help the submitter
(assuming that he takes the advice to work against HEAD not 8.4.x).
But really what we need to see is design documentation, not code.
        regards, tom lane