note to reviewers: reply to the original email
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:
- 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
On Mon, Dec 2, 2013 at 1:30 PM, Peter Eisentraut <peter_e@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
(/messages/by-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.klaver@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
On Mon, Dec 02, 2013 at 04:30:05PM -0500, Peter Eisentraut 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:
+1
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.
You can download a single message as an mbox by clicking "view raw" in the web
archives. Using any email client supporting that format, you can then respond
to the message as though you had received it through a list subscription.
--
Noah Misch
EnterpriseDB http://www.enterprisedb.com
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers