|From:||Simon Riggs <simon(at)2ndQuadrant(dot)com>|
|To:||Andrew Dunstan <andrew(at)dunslane(dot)net>|
|Subject:||Re: Need more reviewers!|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On Fri, 2008-09-05 at 09:19 -0400, Andrew Dunstan wrote:
> Simon Riggs wrote:
> > On Thu, 2008-09-04 at 10:45 -0700, Josh Berkus wrote:
> >> If you are a postgresql hacker at all, or even want to be one, we need your
> >> help reviewing patches! There are several "easy" patches in the list, so
> >> I can assign them to beginners.
> > It would be a reasonable rule that all patch submitters also have to do
> > patch reviews. If we made it a strict rule, then sponsoring companies
> > would know that they *must* provide money/time for that aspect also.
> > Otherwise it is almost impossible to get formal approval to do that.
> All this would do is to deter people from submitting patches. Hard rules
> like this don't work in FOSS communities. I know it's like herding cats,
> but persuasion is really our only tool.
I don't *want* the rule, I just think we *need* the rule because
otherwise sponsors/managers/etc make business decisions to exclude that
aspect of the software dev process.
Otherwise we have a patch-and-dump culture that is unsustainable because
a few people's benevolence as reviewers turns everything into a
bottleneck. It doesn't need to mean loss of control for core and
Simon Riggs www.2ndQuadrant.com
PostgreSQL Training, Services and Support
|Next Message||Peter Eisentraut||2008-09-05 13:40:32||Re: pg_regress inputdir|
|Previous Message||Andrew Dunstan||2008-09-05 13:19:26||Re: Need more reviewers!|