I've posted a patch that would change how the graph server sends email
when a performance *improvement* is detected:
https://bugzilla.mozilla.org/show_bug.cgi?id=904250
Emails about regressions would be unchanged. Emails about improvements
would now be sent to individual patch authors using the same logic as
regression emails:
* If there are up to five authors in the regression range, email
dev-tree-management and each of the patch authors.
* If there are more than five authors, just email dev-tree-management.
I've spoken to other developers in a few places, and most were in favor
of this change. Those who don't want the extra emails can filter them
out if they want to. Motivations for this change:
- When you are expecting a patch to affect performance, the email is
useful confirmation. Receiving it directly saves you the work of
searching through the mailing list archives or manually inspecting graphs.
- When you're not expecting a performance improvement, it may be
important to know that one has happened so you can figure out why.
(Sometimes it's a sign of a bug, for example if you accidentally
disabled some code.)
- Warm fuzzy feeling for developers who write perf wins!
I think the accuracy of the analysis script is now good enough that this
would not generate too many spurious emails. If you have any opinions
or suggestions about this change, please share them here or in bug 904250!
_______________________________________________
dev-platform mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-platform
