-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51647/
-----------------------------------------------------------

(Updated Sept. 26, 2016, 2:32 p.m.)


Review request for Ambari, DIPAYAN BHOWMICK, Pallav Kulshreshtha, and Sreenath 
Somarajapuram.


Changes
-------

Patch updated with review comments.


Bugs: AMBARI-18313
    https://issues.apache.org/jira/browse/AMBARI-18313


Repository: ambari


Description
-------

XML diff tool to show changes made.
View capacity scheduler XML.
Queue capacity chart - chart showing capacities of queue using sunburst chart.
Bug fixes.


Diffs (updated)
-----

  contrib/views/capacity-scheduler/src/main/resources/ui/app/components.js 
8dd7108 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/labelCapacityBar.js
 7776452 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/queueBadge.js
 f8978c2 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/queueHierarchy.js
 64a28343 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/queueMapping.js
 107b377 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/queueSummary.js
 2690c40 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/sunburstChart.js
 PRE-CREATION 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/xmldiffViewer.js
 PRE-CREATION 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/advanced.js
 c079079 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/capsched.js
 447cae0 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/editqueue.js
 7c62ef0 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queuesconf.js
 6a6717a 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/scheduler.js
 40e3e24 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/models/queue.js 
0f68a10 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/router.js fd996ae 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less
 bd875ba 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/templates.js 
2c9609f 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched.hbs
 14625c1 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/advanced.hbs
 5a54d85 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/accessControlList.hbs
 922e27e 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/labelCapacity.hbs
 873fb6f 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/preemption.hbs
 d594c7e 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/queueCapacity.hbs
 608758f 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/queueResources.hbs
 809c1c0 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/queuesconf.hbs
 0c70ee5 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/scheduler.hbs
 66ef1ad 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueHierarchy.hbs
 fb34d51 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueMapping.hbs
 33540e9 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueSummary.hbs
 a7ae9b0 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/sunburstChart.hbs
 PRE-CREATION 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/xmldiffViewer.hbs
 PRE-CREATION 
  contrib/views/capacity-scheduler/src/main/resources/ui/bower.json 911856f 

Diff: https://reviews.apache.org/r/51647/diff/


Testing
-------

Manual unit testing


Thanks,

Akhil PB

Reply via email to