Re: Kudos for Reviewers -- straw poll

On 06/26/2013 12:02 PM, Alvaro Herrera wrote:
> 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.

What about the idea that reviewers who do code revision work, like in
your FK patch, get listed after the original patch author with the
patch, and reviewers do more lightweight reviews get listed at the
bottom of the release notes?  Seems fair to me.

--
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com