Thread: Initial Review Assignments

Initial Review Assignments

From
Josh Berkus
Date:
Reviewers,

Below are your initial assignments for review.  If you can't review the
assigned patch ... or at least, can't review it by Friday ... please let
us know on this list immediately so that it can be assigned to someone else.

Some assignments are marked "TEST".  For those assignments, we're
primarily concerned with you making sure the code runs on one or more
platforms and conforms to the documented API, and do not expect you to
look at the code.

After you have concluded your review, please post your assessment to
pgsql-hackers, update the wiki, and request another patch for review on
this list.

If you are not assigned a patch for review, it is because I didn't know
you are available.  Please let me know.

All patch names below are pastes from the wiki page, so just search the
page (http://wiki.postgresql.org/wiki/CommitFest:2008-11) for that patch.

Also, if anyone is available to review Windows patches, please let me know.

=============

Jaime Casanova: Auto Partitioning Patch

Brendan Jurd: new correlation metric

Robert Haas: Can you take on Zdenek's Upgrade-in-Place patches?

Alex Hunsaker: pre-MED

Ibrar Ahmed: pg_dump roles support

Jeff Davis: contrib/auto_explain

Kenneth Marshall: Tests citext casts

Matthew Wetmore: contrib/pg_stat_statements

Gianni Colli: regexp support in usermaps


Thanks again, and when 8.4 goes out on time, it will be because of you!

--Josh Berkus

Re: [pgsql-rrreviewers] Initial Review Assignments

From
"Robert Haas"
Date:
> Robert Haas: Can you take on Zdenek's Upgrade-in-Place patches?

I can look at 'em, but I'm not sure I know enough to be useful.  I'll
give it a shot, anyway.

...Robert

Re: [pgsql-rrreviewers] Initial Review Assignments

From
"Brendan Jurd"
Date:
On Mon, Nov 3, 2008 at 11:00 AM, Josh Berkus <josh@agliodbs.com> wrote:
> Brendan Jurd: new correlation metric
>

The new correlation metric patch isn't ready for review.  It's
currently undergoing a rethink.  I've moved it to Returned with
Feedback, and the author understands that it might not be a candidate
for 8.4.

I'm available for a new assignment.

Cheers,
BJ

new assignment for Brendan

From
Josh Berkus
Date:
Brendan,

> The new correlation metric patch isn't ready for review.  It's
> currently undergoing a rethink.  I've moved it to Returned with
> Feedback, and the author understands that it might not be a candidate
> for 8.4.
>
> I'm available for a new assignment.

Great.  You want to start on:
SQL Standard Interval output and IntervalStyle GUC
... and move on to its successor patches if it checks out ok?

Thanks!

--Josh


Re: new assignment for Brendan

From
"Brendan Jurd"
Date:
On Tue, Nov 4, 2008 at 12:32 PM, Josh Berkus <josh@agliodbs.com> wrote:
> Great.  You want to start on:
> SQL Standard Interval output and IntervalStyle GUC
> ... and move on to its successor patches if it checks out ok?
>

On it.

Cheers,
BJ