> On April 4, 2016, 10:45 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueSummary.hbs, > > line 60 > > <https://reviews.apache.org/r/44704/diff/1/?file=1295742#file1295742line60> > > > > Can use bunded conditional class names.
This got changed in later commits. I guess I would face conflicts in future patches. Timebeing I will do this. - Akhil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review126808 ----------------------------------------------------------- On April 13, 2016, 7:18 a.m., Akhil PB wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44704/ > ----------------------------------------------------------- > > (Updated April 13, 2016, 7:18 a.m.) > > > Review request for Ambari and Sreenath Somarajapuram. > > > Bugs: AMBARI-15384 > https://issues.apache.org/jira/browse/AMBARI-15384 > > > Repository: ambari > > > Description > ------- > > Revamping the old capsched UI into new better one > > > Diffs > ----- > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components.js > ac679b7 > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/queueHierarchy.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/queueMapping.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/queueSummary.js > PRE-CREATION > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers.js > 16b7d0a > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/advanced.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/capsched.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/editqueue.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queuesconf.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/scheduler.js > PRE-CREATION > contrib/views/capacity-scheduler/src/main/resources/ui/app/helpers.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/helpers/queueMappingParser.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/helpers/uppercase.js > PRE-CREATION > contrib/views/capacity-scheduler/src/main/resources/ui/app/initialize.js > 05bb64c > contrib/views/capacity-scheduler/src/main/resources/ui/app/router.js > a2d50b9 > > contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less > be79229 > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates.js > 25e2aa9 > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/advanced.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/accessControlList.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/editQueueCapacity.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/queueCapacity.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/partials/queueResources.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/queuesconf.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/queuesconf/editqueue.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/scheduler.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueHierarchy.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueMapping.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueSummary.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/versionsPanel.hbs > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/views/editQueueCapacity.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/views/editqueue.js > PRE-CREATION > > contrib/views/capacity-scheduler/src/main/resources/ui/app/views/queuesconf.js > PRE-CREATION > > Diff: https://reviews.apache.org/r/44704/diff/ > > > Testing > ------- > > Manual unit testing > > > Thanks, > > Akhil PB > >