Re: Kudos for Reviewers -- straw poll

Bruce Momjian escribió:
> On Wed, Jun 26, 2013 at 03:22:06PM -0300, Rodrigo Gonzalez wrote:

> > Checking release notes for 9.2.4
> > 
> > you have Fix insecure parsing of server command-line switches
> > (Mitsumasa Kondo, Kyotaro Horiguchi)
> > 
> > What about (it people think that it is good) a second () with reviewed
> > by <someone>....
> 
> That's what we had, and people didn't like it.  If we overload that list
> of names, we might find we want to remove all the names.

Yeah, it becomes too long.  (For security patches, in particular, it's
probably not wise to list reviewers; there might well be reviewers whose
input you never see because they happened in the closed security list).


See the entry for foreign key locks:
 Prevent non-key-field row updates from locking foreign key rows (Álvaro Herrera, Noah Misch, Andres Freund, Alexander
Shulgin,Marti Raudsepp)
 

I am the author of most of the code, yet I chose to add Noah and Andres
because they contributed a huge amount of time to reviewing the patch,
and Alex and Marti because they submitted some code.  They are all
listed as coauthors, which seems a reasonable compromise to me.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services