Re: [RRR] [HACKERS] Commitfest: The Good, The Bad, and the Ugly

2010-09-28 Thread Robert Haas
On Tue, Sep 28, 2010 at 9:33 PM, Itagaki Takahiro wrote: > On Wed, Sep 29, 2010 at 10:18 AM, Robert Haas 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

Re: [RRR] [HACKERS] Commitfest: The Good, The Bad, and the Ugly

2010-09-28 Thread Itagaki Takahiro
On Wed, Sep 29, 2010 at 10:18 AM, Robert Haas 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

Re: [RRR] [HACKERS] Commitfest: The Good, The Bad, and the Ugly

2010-09-28 Thread Robert Haas
On Tue, Sep 28, 2010 at 9:11 PM, Itagaki Takahiro wrote: > On Wed, Sep 29, 2010 at 6:03 AM, David Fetter wrote: >> The Good: >> - Most patches still in play have a reviewer. > > As far as I remember, there were discussions about the issue > "A patch has a reviewer, but in Needs Review state for s

Re: [HACKERS] Commitfest: The Good, The Bad, and the Ugly

2010-09-28 Thread Itagaki Takahiro
On Wed, Sep 29, 2010 at 6:03 AM, David Fetter wrote: > The Good: > - Most patches still in play have a reviewer. As far as I remember, there were discussions about the issue "A patch has a reviewer, but in Needs Review state for several weeks " in 9.0 development. Do we have any plans for it? Ac

[HACKERS] Commitfest: The Good, The Bad, and the Ugly

2010-09-28 Thread David Fetter
Folks, We're almost half way through the commitfest, and so I'll start with: The Good: - Most patches still in play have a reviewer. - It's possible for one person to post 5 reviews in a day. Robert Haas actually did this on his own time yesterday. - New people have been reviewing patches,