----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12880/ -----------------------------------------------------------
(Updated July 26, 2013, 6:23 p.m.) Review request for mesos. Changes ------- Moved `computeFrameworkStats` next to where it is called. Repository: mesos-git Description (updated) ------- Compute framework stats in one iteration Rather than iterate over 'frameworks' and 'completed_frameworks' once to update the slave mappings and a second time to calculate stats for the frameworks, move that work into a single loop. This also removes a 'concat' call that was creating a temporary Array used to calculate stats once and then thrown away. Review: http://reviews.apache.org/r/12880 Diffs (updated) ----- src/webui/master/static/js/controllers.js a757f131b37053eb0dc81d2f057b233cef5ecc57 Diff: https://reviews.apache.org/r/12880/diff/ Testing ------- Rendered /slaves/:id to verify framework stats were still calculated properly. Thanks, Ross Allen