Re: [9.4 CF 1] The Commitfest Slacker List

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Josh Berkus <josh(at)agliodbs(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgreSQL(dot)org>
Subject: Re: [9.4 CF 1] The Commitfest Slacker List
Date: 2013-06-24 12:13:14
Message-ID: 51C837DA.3040307@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 06/24/2013 12:41 AM, Josh Berkus wrote:
> Folks,
>
> For 9.2, we adopted it as policy that anyone submitting a patch to a
> commitfest is expected to review at least one patch submitted by someone
> else. And that failure to do so would affect the attention your patches
> received in the future. For that reason, I'm publishing the list below
> of people who submitted patches and have not yet claimed any patch in
> the commitfest to review.
>
> For those of you who are contributing patches for your company, please
> let your boss know that reviewing is part of contributing, and that if
> you don't do the one you may not be able to do the other.
>
> The two lists below, idle submitters and committers, constitutes 26
> people. This *outnumbers* the list of contributors who are busy
> reviewing patches -- some of them four or more patches. If each of
> these people took just *one* patch to review, it would almost entirely
> clear the list of patches which do not have a reviewer. If these folks
> continue not to do reviews, this commitfest will drag on for at least 2
> weeks past its closure date.
>
> Andrew Geirth
> Nick White
> Peter Eisentrout
> Alexander Korotkov
> Etsuro Fujita
> Hari Babu
> Jameison Martin
> Jon Nelson
> Oleg Bartunov
> Chris Farmiloe
> Samrat Revagade
> Alexander Lakhin
> Mark Kirkwood
> Liming Hu
> Maciej Gajewski
> Josh Kuperschmidt
> Mark Wong
> Gurjeet Singh
> Robins Tharakan
> Tatsuo Ishii
> Karl O Pinc
>
> Additionally, the following committers are not listed as reviewers on
> any patch. Note that I have no way to search which ones might be
> *committers* on a patch, so these folks are not necessarily slackers
> (although with Bruce, we know for sure):
>
> Bruce Momjian (momjian)
> Michael Meskes (meskes)
> Teodor Sigaev (teodor)
> Andrew Dunstan (adunstan)
> Magnus Hagander (mha)
> Itagaki Takahiro (itagaki)
>

I think we maybe need to be a bit more careful about a name and shame
policy, or it will be ignored. I actually reviewed Peter's emacs config
patch just yesterday, although I didn't note that on the Commitfest app.
(Incidentally, based on Tom's later comments I think we should probably
mark that one as rejected.)

Anyway, I have claimed the VPATH patches for review, and will look at
them today.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2013-06-24 12:17:51 Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review]
Previous Message Andres Freund 2013-06-24 12:12:34 Re: preserving forensic information when we freeze