Re: Feature patch 1 for plperl [PATCH] - Mailing list pgsql-hackers

From Robert Haas
Subject Re: Feature patch 1 for plperl [PATCH]
Date
Msg-id 603c8f071001121606u788dfd4apf8b2ec521b0507dd@mail.gmail.com
Whole thread Raw
In response to Re: Feature patch 1 for plperl [PATCH]  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: Feature patch 1 for plperl [PATCH]  (Tim Bunce <Tim.Bunce@pobox.com>)
List pgsql-hackers
On Sun, Jan 10, 2010 at 4:35 PM, Robert Haas <robertmhaas@gmail.com> wrote:
>>> I would strongly suggest to Tim that he rip the portions of this patch
>>> that are related to this feature out and submit them separately so
>>> that we can commit the uncontroversial portions first.
>>
>> See my previous email. I suggested that Tim send three patches: one for this
>> controversial stuff, one for the new utility functions for plperl, and one
>> for the remainder. He and I have discussed it and I believe he is agreeable
>> to that.
>
> OK, well then just +1 for that.

I believe we have agreement on this course of action, so I'm going to
mark the current patch as Returned with Feedback.  Hopefully Tim will
submit separate patches for each of these three areas in the next day
or two before start-of-CommitFest - personally, I think they should
each get their own thread and their own entry in the CommitFest app,
for ease of tracking and reviewing.  YMMV, of course.

...Robert


pgsql-hackers by date:

Previous
From: "Kevin Grittner"
Date:
Subject: Re: Any ExecStoreTuple end runs?
Next
From: "Joshua D. Drake"
Date:
Subject: Re: mailing list archiver chewing patches