Re: Feature patch 1 for plperl [PATCH]

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

In response to

Responses

Browse pgsql-hackers by date

  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?