| From: | Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | David Fetter <david(at)fetter(dot)org>, PostgreSQL General <pgsql-general(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org>, Round Robin Reviewers <pgsql-rrreviewers(at)postgresql(dot)org> |
| Subject: | Re: [RRR] [HACKERS] Commitfest: The Good, The Bad, and the Ugly |
| Date: | 2010-09-29 01:33:21 |
| Message-ID: | AANLkTimmV--pHOxEZ9NatFfvt2zngxiBj-2CH-2mLbi+@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-general pgsql-hackers pgsql-rrreviewers |
On Wed, Sep 29, 2010 at 10:18 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> No, the column is very clearly labelled "Reviewers", not "Reviewer".
> And we have certainly had patches with more than one person's name in
> that field in the past. The issue is rather that we don't have enough
> people reviewing. We haven't had enough people volunteer to do
> reviews to even assign ONE person to each patch, let alone two. There
> are, as of this writing, SEVEN patches that have no reviewer at all.
Some of them might be too difficult to review. For example, replication
or snapshot management requires special skills to review.
I'm worrying about new reviewers hesitate to review a patch that has
a previous reviewer, and then, if they think the remaining patches are
too difficult for them, they would just leave the commitfest page.
--
Itagaki Takahiro
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2010-09-29 01:58:10 | Re: Restore problem from 8.4 backup to 9.0 |
| Previous Message | Robert Haas | 2010-09-29 01:18:27 | Re: [RRR] [HACKERS] Commitfest: The Good, The Bad, and the Ugly |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Darren Duncan | 2010-09-29 01:43:18 | Re: Proposal: plpgsql - "for in array" statement |
| Previous Message | Darren Duncan | 2010-09-29 01:31:11 | Re: Proposal: plpgsql - "for in array" statement |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andy Colson | 2010-09-29 02:15:28 | Re: [RRR] [HACKERS] Commitfest: The Good, The Bad, and the Ugly |
| Previous Message | Robert Haas | 2010-09-29 01:18:27 | Re: [RRR] [HACKERS] Commitfest: The Good, The Bad, and the Ugly |