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
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 |