From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Tim Bunce <Tim(dot)Bunce(at)pobox(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Feature patch 1 for plperl [PATCH] |
Date: | 2010-01-13 00:06:59 |
Message-ID: | 603c8f071001121606u788dfd4apf8b2ec521b0507dd@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Jan 10, 2010 at 4:35 PM, Robert Haas <robertmhaas(at)gmail(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | J. Greg Davidson | 2010-01-13 00:33:35 | xml2 still essential for us |
Previous Message | Kevin Grittner | 2010-01-12 23:48:49 | Re: Any ExecStoreTuple end runs? |