Re: next CommitFest

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Albert Cervera i Areny <albert(at)nan-tic(dot)com>, pgsql-hackers(at)postgresql(dot)org, Euler Taveira de Oliveira <euler(at)timbira(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>
Subject: Re: next CommitFest
Date: 2009-11-13 04:31:33
Message-ID: 200911130431.nAD4VXu21374@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Robert Haas wrote:
> >> Personally, I would not propose to impose this rule of first-time
> >> contributors, or even second-time contributors. ?But by about patch #3
> >> I think everyone should be pitching in.
> >
> > I hate to ask, but how would we enforce this? ?Do we no longer apply
> > patches for 3rd-time submitters who have not reviewed? ?That seems to be
> > hurting us more than them. ? Are we prepared to discard valid patches
> > for this reason?
>
> We just wouldn't assign round-robin reviewers to such patches. If
> someone wants to volunteer, more power to them, but we would encourage
> people to focus their efforts on the patches of people who were
> themselves reviewing. It's important to keep in mind that "valid" is
> not a boolean. Some patches are perfect the day they roll in, but not
> too many. It takes work to get them committable, and I don't see why
> anyone should have an expectation that they can have that help for
> themselves without doing the same thing for other people.

OK, but the problem I see there is that the reviewers are there to
assist the committers; if no one reviews something, it just makes more
work for the committers.

> All that having been said, the real shortage ATM is of committers
> rather than reviewers. We have plenty of them, but many of them
> commit almost nothing. I don't want to minimize the contributions of
> the non-Tom committers, but Tom is numerically far and away committing
> more than anyone else, and not small patches, either. Beyond the
> fact that it makes the CommitFest slow, long, and not too much fun for
> Tom, it also means that Tom has less time available to do things that
> Only Tom Can Do. I venture to say that there will be Great Excitement
> about the enhancements to the EPQ machinery and PL/pgsql that Tom has
> recently effected. Well, if Tom hadn't had to single-handedly handle
> so many patches last CF, maybe he would have done something else cool,
> too.

Totally agree.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message KaiGai Kohei 2009-11-13 04:32:36 [PATCH] Largeobject Access Controls (r2432)
Previous Message Robert Haas 2009-11-13 04:20:12 CTE containing ambiguous columns