CommitFest 2011-11 Post-Tryptophan Progress Report

With plenty of people in the US returning to work on Monday after a
modest break last week, I wanted to put out an updated on the active
CommitFest. I just sent out several pleas for help with specific patches
where only a limited chunk of the community is really in a good position
to test the code. Excluding those for the moment, here are some notes on
the submitted patches that not only don't have a reviewer already, I
don't know who might take them on yet:

"Command Triggers" by Dimitri Fontaine. This is a pretty big patch (3366
lines) that won't be easy to slug through. I know Dimitri is still
making some progress on this just by thinking more about the code, but
that's not going to keep up for much longer without outside feedback.

"type privileges" by Peter Eisentraut. Another big patch (2537 lines),
and it needs both some code review and searching for security issues in
the new feature.

Both "type privledges" and "Command Triggers" are likely to slip well
into December if we don't get a major reviewer working on them soon. I
just don't know who that might be in either case.

"cursor calling with named parameters" by Yeb Havinga. This is a
resubmission of a patch from the last CommitFest, so that previous
feedback is around to give an idea what should be re-tested in the new
version. At 702 lines including docs and tests, I'd call this a medium
sized patch to work on. It'd say it's a decent candidate for someone who
picked up a patch, got done early, and is enough of a glutton to want to
do a second patch.

"splitting plpython into smaller parts" by Jan Urbański. Just asked for
clarification on this one because I'm not quite sure what to do with it.

"unite recovery.conf and postgresql.conf": Since this has a lot more
consensus issues than code ones, I'm going to look into this more
myself. The things I submitted actually change around the potential ways
to satisfy all the use cases here, too.

"Configuration include directory" and "includeifexists in configuration
file" by me. These are not too hard to look at, and given that this all
started as hacking on Mangus's code he may get to it eventually anyway.
I'm not too worried about these two right now.

Hi Greg,

On Monday, November 28, 2011 11:13:52 AM Greg Smith wrote:
> With plenty of people in the US returning to work on Monday after a
> modest break last week, I wanted to put out an updated on the active
> CommitFest. I just sent out several pleas for help with specific patches
> where only a limited chunk of the community is really in a good position
> to test the code. Excluding those for the moment, here are some notes on
> the submitted patches that not only don't have a reviewer already, I
> don't know who might take them on yet:
>
> "Command Triggers" by Dimitri Fontaine. This is a pretty big patch (3366
> lines) that won't be easy to slug through. I know Dimitri is still
> making some progress on this just by thinking more about the code, but
> that's not going to keep up for much longer without outside feedback.

> Both "type privledges" and "Command Triggers" are likely to slip well
> into December if we don't get a major reviewer working on them soon. I
> just don't know who that might be in either case.
Unless a major reviewer cries, I will - although not a major reviewer - take a
stab thursday if thats fine with you.

Andres

On 11/28/2011 05:38 PM, Andres Freund wrote:
> Unless a major reviewer cries, I will - although not a major reviewer
> - take a
> stab thursday if thats fine with you.

I intended to use that term to suggest the review job itself could
become a major chunk of work, not to try and classify reviewers. Bad
wording choice. If you could take a look at Command Triggers later this
week I'm sure Dimitri would appreciate your feedback.

On Tuesday, November 29, 2011 02:45:50 AM Greg Smith wrote:
> On 11/28/2011 05:38 PM, Andres Freund wrote:
> > Unless a major reviewer cries, I will - although not a major reviewer
> > - take a
> > stab thursday if thats fine with you.
>
> I intended to use that term to suggest the review job itself could
> become a major chunk of work, not to try and classify reviewers. Bad
> wording choice. If you could take a look at Command Triggers later this
> week I'm sure Dimitri would appreciate your feedback.
Well. I don't think its that bad wording... Imo the patch will soon need input
from a commiter with experience in the area to solidify the direction...

Thanks,

Andres