Re: [9.4 CF 1] The Commitfest Slacker List

On 04/07/13 10:43, Robert Haas wrote:

> And
> people who submit patches for review should also review patches: they
> are asking other people to do work, so they should also contribute
> work.
>

I think that is an overly simplistic view of things. People submit
patches for a variety of reasons, but typically because they think the
patch will make the product better (bugfix or new functionality). This
is a contribution in itself, not a debt.

Now reviewing is performed to ensure that submitted code is *actually*
going to improve the product.

Both these activities are volunteer work - to attempt to tie them
together forceably is unusual to say the least. The skills and
experience necessary to review patches are considerably higher than
those required to produce patches, hence the topic of this thread.

Now I do understand we have a shortage of reviewers and lots of patches,
and that this *is* a problem - but what a wonderful problem...many open
source projects would love to be in this situation!!!

Regards

Mark