When someone submits a patch for review, gerrit is nice enough to email people, but the email only contains misc metadata about the patch. Often it is not clear from the commit message whether the patch is something I should take a closer look at. Switching from email over to the web UI to skim the patch is not a very efficient workflow. I would prefer to be able to see the full commit diff in the initial email alert, so I can very quickly see if the code is worth taking a closer look at. Only then would I go to gerrit for a detailed review.
I can understand some people would not want have their email "bloated" with patches, so I imagine some email notification preference in gerrit would be required to turn patch diff inclusion on/off. Oh, and the diff should be inline to the mail email body, not an attachment.
This appears to be implemented in upstream gerrit:
4a80e99 Add unified diff to newchange mail template.
02aacbc sendemail.
I second the notion that this would be useful (and more useful if the email could then be replied to as mentioned in https:/