search

andymeneely / chromium-history

Scripts and data related Chromium's history
11 stars 4 forks source link

Max Churn methods on Code Review #96

Closed andymeneely closed 10 years ago

andymeneely commented 10 years ago

Currently total_churn will do a join on PatchSet and PatchSetFiles and add up everything. But that might not be the best representative of the workload that a given code review has. I'm thinking the size of the largest patch set is a better representative. So we need to do a sum of PatchSetFiles, grouped by PatchSet, then the Max of num_added and num_removed.

andymeneely commented 10 years ago

Finished with af7cdcb5e44679016ab963deb04d967133c1d169