mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrei Sekretenko <>
Subject Re: Review Request 71178: Implemented displaying roles of multi-role frameworks as a tree.
Date Fri, 23 Aug 2019 17:34:07 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Aug. 23, 2019, 5:34 p.m.)

Review request for mesos, Benjamin Mahler and Meng Zhu.

Bugs: MESOS-8503

Repository: mesos


This patch makes the UI pages `frameworks` and `framework` display
roles of multi-role frameworks as collapsible tree (instead of a list).

Diffs (updated)

  src/webui/app/app.js f6f11384f35b260dcaff432c884dc063ea5e0f0e 
  src/webui/app/frameworks/framework.html 82f6b279a9416d147fcfab094a326d67d0951dcc 
  src/webui/app/frameworks/frameworks.html d37c6137b638a27e5bd0f70f08733d81550b3ace 
  src/webui/app/frameworks/roles-tree-root.html PRE-CREATION 
  src/webui/app/frameworks/roles-tree.html PRE-CREATION 
  src/webui/app/frameworks/roles.html PRE-CREATION 
  src/webui/assets/css/mesos.css 0ff47cda36cc897f2e8f43804d38046f3e27e575 




Tested manually with conbination of frameworks with thousands of roles and frameworks with
1-2 roles.

Performance: re-rendering time of `frameworks` page with 50 frameworks with ~4000 roles each
is around 500 ms on my hardware. 
With all subtrees collapsed, more than half of this time is spent inside `intermediateRoleTree()`
and `aggregateRoleTree()` (roughly equal amounts of time).
If/when the master starts to store the roles as a tree, it might make sense to convert this
to getting roles in the tree form.


Andrei Sekretenko

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message