Thread: CommitFest 2010-11: Call for Reviewers

CommitFest 2010-11: Call for Reviewers

From
Greg Smith
Date:
It's the time of year again where leaves are falling, temperatures
dropping, and patches cry out looking for one last review before the
start of the holiday season.  The patches submitted so far are listed at
https://commitfest.postgresql.org/action/commitfest_view?id=8 ,
featuring a mix of brand new ones with some going through their second
or third round of rework.  Expect to see a furious weekend of last
minute patches that arrive just before the deadline too.  The official
cut-off to start the CommitFest is Mon Nov 15 00:00:00 UTC.

If you're interested in reviewing a patch but haven't done so before,
the process is outlined at the following page:

http://wiki.postgresql.org/wiki/Reviewing_a_Patch
http://wiki.postgresql.org/wiki/RRReviewers

All you have to do to "claim" a patch is update the CommitFest
application page to put your name down as the reviewer, then do the
review in a timely fashion.  Look for the patches with the bright red
"Nobody" listed in the reviewer field.

If you have questions about which patch to work on, please try to keep
e-mail traffic related to topics like patch selection and volunteering
for a new round-robin assignment to the pgsql-rrreviewers mailing list.
Whereas the pgsql-hackers list is the right destination for the actual
review itself.

--
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD



Re: CommitFest 2010-11: Call for Reviewers

From
David Fetter
Date:
On Fri, Nov 12, 2010 at 07:08:15PM -0500, Greg Smith wrote:
> It's the time of year again where leaves are falling, temperatures
> dropping, and patches cry out looking for one last review before the
> start of the holiday season.  The patches submitted so far are
> listed at
> https://commitfest.postgresql.org/action/commitfest_view?id=8 ,
> featuring a mix of brand new ones with some going through their
> second or third round of rework.  Expect to see a furious weekend of
> last minute patches that arrive just before the deadline too.  The
> official cut-off to start the CommitFest is Mon Nov 15 00:00:00 UTC.
>
> If you're interested in reviewing a patch but haven't done so
> before, the process is outlined at the following page:
>
> http://wiki.postgresql.org/wiki/Reviewing_a_Patch
> http://wiki.postgresql.org/wiki/RRReviewers
>
> All you have to do to "claim" a patch is update the CommitFest
> application page to put your name down as the reviewer, then do the
> review in a timely fashion.  Look for the patches with the bright
> red "Nobody" listed in the reviewer field.
>
> If you have questions about which patch to work on, please try to
> keep e-mail traffic related to topics like patch selection and
> volunteering for a new round-robin assignment to the
> pgsql-rrreviewers mailing list.  Whereas the pgsql-hackers list is
> the right destination for the actual review itself.

There's also an IRC channel

irc://irc.freenode.net/pg_charlie_foxtrot

Cheers,
David.
--
David Fetter <david@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

Re: CommitFest 2010-11: Call for Reviewers

From
Greg Smith
Date:
David Fetter wrote:
> There's also an IRC channel
> irc://irc.freenode.net/pg_charlie_foxtrot
>

What's the difference between this and the #postgresql channel on
freenode?  This is the first I've heard of this channel.

--
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services and Support        www.2ndQuadrant.us



Re: CommitFest 2010-11: Call for Reviewers

From
Greg Smith
Date:
Despite some jerk submitting three brand new patches at the last minute
by himself, the November CommitFest is now closed for new submissions
and ready for review!  Only 7 of the 40 patches that need review have a
reviewer assigned so far.  Reviewers, mark down a patch you want to look
at in the CommitFest app:
https://commitfest.postgresql.org/action/commitfest_view?id=8 or ask to
be assigned one on the pgsql-rrreviewers list, if you want to work on
something but don't have a preference for which patch.  There are a few
larger patches in this round, but the majority of those already have a
reviewer attached to them.  Many of the remaining patches are pretty
approachable even if you're relatively new to reviewing PostgreSQL
code.  See the usual for details:

http://wiki.postgresql.org/wiki/Reviewing_a_Patch
http://wiki.postgresql.org/wiki/RRReviewers

--
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services and Support        www.2ndQuadrant.us



Re: CommitFest 2010-11: Call for Reviewers

From
David Fetter
Date:
On Sun, Nov 14, 2010 at 08:02:46PM -0500, Greg Smith wrote:
> David Fetter wrote:
> >There's also an IRC channel
> >irc://irc.freenode.net/pg_charlie_foxtrot
>
> What's the difference between this and the #postgresql channel on
> freenode?  This is the first I've heard of this channel.

The #postgresql channel on freenode can get very noisy.  It averages
about 500 people, which means it's tough to have a laggy conversation
about almost anything, which frequently happens in small interest
groups like RRReviewers.

Cheers,
David.
--
David Fetter <david@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

Re: CommitFest 2010-11: Call for Reviewers

From
Greg Smith
Date:
At two days in, we're still at a pretty low number of reviewers so far.
There are still 18 patches that don't have a reviewer at all yet.
(Rattles coin cup)  Can you spare a little time to help out PostgreSQL?

--
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services and Support        www.2ndQuadrant.us



Re: CommitFest 2010-11: Call for Reviewers

From
Bernd Helmle
Date:

--On 17. November 2010 18:05:43 -0500 Greg Smith <greg@2ndquadrant.com>
wrote:

> At two days in, we're still at a pretty low number of reviewers so far.
> There are still 18 patches that don't have a reviewer at all yet.
> (Rattles coin cup)  Can you spare a little time to help out PostgreSQL?

Unfortunately i'm completely out of time to help out, at least until end of
next week. I've planned to jump in beginning December when most of my stuff
here is done, but of course this doesn't help very much now :(

        Bernd




Re: CommitFest 2010-11: Call for Reviewers

From
Jeff Janes
Date:
On Wed, Nov 17, 2010 at 3:05 PM, Greg Smith <greg@2ndquadrant.com> wrote:
> At two days in, we're still at a pretty low number of reviewers so far.
>  There are still 18 patches that don't have a reviewer at all yet.  (Rattles
> coin cup)  Can you spare a little time to help out PostgreSQL?

Hi Greg,

I'll give it a go.  Not the Windows or JDBC ones though, please.

Cheers,

Jeff