-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33056/
-----------------------------------------------------------
(Updated April 10, 2015, 9:58 p.m.)
Review request for Ambari, Srimanth Gunturi, Xi Wang, and Yusaku Sako.
Changes
-------
Updated patch changes the metric selection link UX and makes it consistent with
others.
Bugs: AMBARI-10427
https://issues.apache.org/jira/browse/AMBARI-10427
Repository: ambari
Description
-------
As part of this ticket service specific heatmap will be shown in their
respective page with a new tab "Heatmaps". (The screenshot is attached on the
Jira. Also the work can be seen on the brunch server with mock data).
Dashboard->Heatmaps will show system metrics heatmap
Apart from this, spinner has been changed to be shown next to the Label of the
heatmap rather than individual div element. Later makes the page go up and down
when the spinner is shown/hidden.
Diffs (updated)
-----
ambari-web/app/assets/data/stacks/HDP-2.1/service_components.json 23f65f2
ambari-web/app/controllers.js 816dbf5
ambari-web/app/controllers/main/charts/heatmap.js ecc57b3
ambari-web/app/controllers/main/service/info/heatmap.js PRE-CREATION
ambari-web/app/messages.js 4d7a811
ambari-web/app/models/stack_service.js 39995bb
ambari-web/app/routes/main.js 5083549
ambari-web/app/styles/application.less d4725f1
ambari-web/app/templates/main/charts/heatmap.hbs e1942bd
ambari-web/app/templates/main/service/item.hbs eed34f2
ambari-web/app/views.js 936b2d3
ambari-web/app/views/main/charts/heatmap.js 59b36ff
ambari-web/app/views/main/charts/heatmap/heatmap_rack.js 11c4f64
ambari-web/app/views/main/service/info/heatmap_view.js PRE-CREATION
ambari-web/app/views/main/service/info/menu.js bdb0e1d
ambari-web/app/views/main/service/item.js db2b490
Diff: https://reviews.apache.org/r/33056/diff/
Testing
-------
Tested on a live cluster, brunch server.
Verified that ambari-web unit tests passes successfully with the patch
Thanks,
Jaimin Jetly
