From: | Brendan Jurd <direvus(at)gmail(dot)com> |
---|---|
To: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Dave Page <dpage(at)pgadmin(dot)org>, Bruce Momjian <bruce(at)momjian(dot)us>, 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> |
Subject: | Re: next CommitFest |
Date: | 2009-11-13 17:32:05 |
Message-ID: | 37ed240d0911130932i3b48849csb8cbae061abf11e4@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2009/11/14 Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>:
> I think we (the commitfest manager?) should simply send polite message
> to any regulars who submits patches but hasn't volunteered for review.
> Along the lines of:
>
I certainly endorse Heikki's suggestion, but I wonder if we can do
more to make reviewing patches an attractive option.
As Tom notes, reviewing somebody else's patch just isn't as much fun
as working on your own. Robert notes that reviewing patches is a
great way to learn about the codebase, and I concur. But in terms of
hacker satisfaction it just doesn't compare to creating something, and
I think the ratios of submitters to reviewers in recent CFs vindicate
me here.
I'm thinking of something like a Reviewer Hall of Fame, or Honour
Roll. During and after a commitfest, it shows how many reviews have
been completed by each person [1].
This could be included in the Weekly News at the CF's conclusion.
One of the things that people get out of contributing to an OSS
project is the recognition of their peers. Well then, let's leverage
that by acclaiming the people who put in a lot of effort reviewing,
loudly and publicly. The louder and more public, the more powerful
the incentive.
Cheers,
BJ
[1] perhaps with some subjective weighting with respect to patch
complexity / depth of review.
From | Date | Subject | |
---|---|---|---|
Next Message | Greg Sabino Mullane | 2009-11-13 17:35:11 | Re: Listen / Notify rewrite |
Previous Message | Florian G. Pflug | 2009-11-13 17:27:59 | Inspection of row types in pl/pgsql and pl/sql |