-----------------------------------------------------------
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

Reply via email to