On Mon, Dec 2, 2013 at 1:30 PM, Peter Eisentraut <pete...@gmx.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@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgsql-hackers -- Adrian Klaver adrian.kla...@gmail.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers