Re: note to reviewers: reply to the original email

On Mon, Dec 2, 2013 at 1:30 PM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
> A note to reviewers participating in the commit fests: When you send a
> review of a patch, reply to the email that contains the patch. Do no
> start a new email thread like "Review of 'Some Patch'". This is
> important for several reasons:

Might want to change the example
(http://www.postgresql.org/message-id/1247855889.6125.6.camel@lapdragon)
in:

https://wiki.postgresql.org/wiki/Reviewing_a_Patch

It shows the behavior you do not want :)

>
> - It maintains the integrity of the email archives. The commit fest app
> is supposed to track discussions, not be a participating link in the
> discussions.
>
> - It makes sure the right people get your review. If you start a new
> thread, worst case, no one participating in the original thread will see
> your email. This is especially true if the original thread and the
> commit fest entry have different titles, and you don't even CC the patch
> author on your review.
>
> If you don't have the original email in your email client, at least
> doctor up the subject line so it looks like a reply, so that email
> clients can do a reasonable job sorting the two threads together. Yes,
> email is stupid.
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers

--
Adrian Klaver
adrian(dot)klaver(at)gmail(dot)com