Review Request 59103: Pig and CapSched views are not loading in 2.4.3

2017-05-09 Thread Akhil PB
f/1/ Testing --- Manual unit testing Thanks, Akhil PB

Re: Review Request 56379: AMBARI-19418: Support setup queue priority in Ambari - Capacity scheduler view

2017-02-09 Thread Akhil PB
--- Manunal Unit Testing Thanks, Akhil PB

Review Request 56379: AMBARI-19418: Support setup queue priority in Ambari - Capacity scheduler view

2017-02-07 Thread Akhil PB
/capacity-scheduler/src/main/resources/ui/app/store.js dce00c1 contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/queue.hbs 87f00d9 Diff: https://reviews.apache.org/r/56379/diff/ Testing --- Manunal Unit Testing Thanks, Akhil PB

Review Request 55555: Ambari Yarn Queue Manager allows two same queue name

2017-01-15 Thread Akhil PB
/resources/ui/app/components/pathInput.js e71d226 Diff: https://reviews.apache.org/r/5/diff/ Testing --- Manual unit testing Thanks, Akhil PB

Re: Review Request 54888: Ambari capacity queue manager only accepts values in integers i.e not decimal values like 0.5

2017-01-10 Thread Akhil PB
/capacity-scheduler/src/main/resources/ui/app/templates/components/editQueueCapacity.hbs baa77db Diff: https://reviews.apache.org/r/54888/diff/ Testing --- Manual unit testing Thanks, Akhil PB

Re: Review Request 54888: Ambari capacity queue manager only accepts values in integers i.e not decimal values like 0.5

2017-01-09 Thread Akhil PB
/views/capacity-scheduler/src/main/resources/ui/app/templates/components/editQueueCapacity.hbs baa77db Diff: https://reviews.apache.org/r/54888/diff/ Testing --- Manual unit testing Thanks, Akhil PB

Re: Review Request 54888: Ambari capacity queue manager only accepts values in integers i.e not decimal values like 0.5

2017-01-06 Thread Akhil PB
/app/styles/application.less e24bf4e contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capacityEditForm.hbs 78aca04 Diff: https://reviews.apache.org/r/54888/diff/ Testing --- Manual unit testing Thanks, Akhil PB

Review Request 54888: Ambari capacity queue manager only accepts values in integers i.e not decimal values like 0.5

2016-12-20 Thread Akhil PB
/main/resources/ui/app/templates/capacityEditForm.hbs 78aca04 Diff: https://reviews.apache.org/r/54888/diff/ Testing --- Manual unit testing Thanks, Akhil PB

Re: Review Request 52688: Capacity Scheduler View: Calculating absolute capacity for node labels and showing in sunburst chart

2016-10-19 Thread Akhil PB
ated e-mail. To reply, visit: https://reviews.apache.org/r/52688/#review152284 ------- On Oct. 19, 2016, 11:42 a.m., Akhil PB wrote: > > --- > This is a

Re: Review Request 52688: Capacity Scheduler View: Calculating absolute capacity for node labels and showing in sunburst chart

2016-10-19 Thread Akhil PB
/src/main/resources/ui/app/templates/components/editLabelCapacity.hbs 8fe627e contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueMapping.hbs f143911 Diff: https://reviews.apache.org/r/52688/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 52688: Capacity Scheduler View: Calculating absolute capacity for node labels and showing in sunburst chart

2016-10-16 Thread Akhil PB
Thanks, Akhil PB

Review Request 52688: Capacity Scheduler View: Calculating absolute capacity for node labels and showing in sunburst chart

2016-10-10 Thread Akhil PB
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueMapping.hbs f143911 Diff: https://reviews.apache.org/r/52688/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-28 Thread Akhil PB
-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

Re: Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-26 Thread Akhil PB
t; > Package versiosn must be fixed. Blob has no versions in bower. Bower itself added '*' as version. - Akhil --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51647/#review150166 --- On Sept. 2

Re: Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-26 Thread Akhil PB
51943db440dbd7c15d515597cda2e1; Changed into as a bower dependency. - Akhil --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51647/#review150165 --------

Re: Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-26 Thread Akhil PB
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

Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-06 Thread Akhil PB
://reviews.apache.org/r/51647/diff/ Testing --- Manual unit testing Thanks, Akhil PB

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-09-01 Thread Akhil PB
/capacity-scheduler/src/main/resources/ui/app/templates/components/queueSummary.hbs 9c3ffd0 Diff: https://reviews.apache.org/r/51452/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-09-01 Thread Akhil PB
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueSummary.hbs 9c3ffd0 Diff: https://reviews.apache.org/r/51452/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 51452: AMBARI-18261: CapSched View: Showing warning icons to refresh or restart capsched once after configs are saved

2016-08-31 Thread Akhil PB
etter to put all finds into a promise array. Will try with promise array! - Akhil --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51452/#review147417 ---------

Re: Review Request 49755: AMBARI-17602: Capacity Scheduler View - Fetching current RM configuration of queues and preemption implementation

2016-08-23 Thread Akhil PB
/resources/ui/app/views/editqueue.js a5d0ef6 Diff: https://reviews.apache.org/r/49755/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 49755: AMBARI-17602: Capacity Scheduler View - Fetching current RM configuration of queues and preemption implementation

2016-08-08 Thread Akhil PB
contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/components/queueSummary.hbs ef5663d contrib/views/capacity-scheduler/src/main/resources/ui/app/views/editqueue.js a5d0ef6 Diff: https://reviews.apache.org/r/49755/diff/ Testing --- Manual Unit Testing Thanks, Akhil

Review Request 50070: Capacity Scheduler View - Fix for nodelabel api compatibility issue in HDP-2.5

2016-07-15 Thread Akhil PB
/views/capacity-scheduler/src/main/resources/ui/app/store.js 4a9c6b0 Diff: https://reviews.apache.org/r/50070/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 49755: Capacity Scheduler View - Fetching current RM configuration of queues and preemption implementation

2016-07-15 Thread Akhil PB
ef5663d contrib/views/capacity-scheduler/src/main/resources/ui/app/views/editqueue.js a5d0ef6 Diff: https://reviews.apache.org/r/49755/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 49755: Capacity Scheduler View - Fetching current RM configuration of queues and preemption implementation

2016-07-14 Thread Akhil PB
-scheduler/src/main/resources/ui/app/views/editqueue.js a5d0ef6 Diff: https://reviews.apache.org/r/49755/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Review Request 49755: Capacity Scheduler View - Fetching current RM configuration of queues and preemption implementation

2016-07-07 Thread Akhil PB
/49755/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-29 Thread Akhil PB
t; Where is the versions-table-wrap class defined? > > Wouldnt it cause the views/queues.js code to break? > > Akhil PB wrote: > versionPanel.hbs itself is a new file and class "versions-table-wrap" is > used for styling and styles are defined in application.les

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-29 Thread Akhil PB
t; Where is the versions-table-wrap class defined? > > Wouldnt it cause the views/queues.js code to break? > > Akhil PB wrote: > versionPanel.hbs itself is a new file and class "versions-table-wrap" is > used for styling and styles are defined in application.les

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-29 Thread Akhil PB
t; Where is the versions-table-wrap class defined? > > Wouldnt it cause the views/queues.js code to break? > > Akhil PB wrote: > versionPanel.hbs itself is a new file and class "versions-table-wrap" is > used for styling and styles are defined in application.les

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-26 Thread Akhil PB
/editQueueCapacity.js b2ae1d5 contrib/views/capacity-scheduler/src/main/resources/ui/app/views/editqueue.js 0dde62b contrib/views/capacity-scheduler/src/main/resources/ui/app/views/queuesconf.js 5cef8e5 Diff: https://reviews.apache.org/r/48487/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-24 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48487/#review139046 ------- On June 9, 2016, 12:05 p.m., Akhil PB wrote: > > -

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-24 Thread Akhil PB
------ On June 9, 2016, 12:05 p.m., Akhil PB wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48487/ > ---

Review Request 49141: Capacity Scheduler View: Javascript error causing new queue creation failed in Yarn Capacity Scheduler

2016-06-23 Thread Akhil PB
Diff: https://reviews.apache.org/r/49141/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-22 Thread Akhil PB
since queues.js uses above as id and versionPanel uses as class name. - Akhil --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48487/#review139036 ------

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-22 Thread Akhil PB
> On June 22, 2016, 8:50 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js, > > line 164 > > <https://reviews.apache.org/r/48487/diff/1/?file=1412286#file1412286line164> > > > > Why ar

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-22 Thread Akhil PB
------ On June 9, 2016, 12:05 p.m., Akhil PB wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48487/ > ---

Re: Review Request 48398: Capacity scheduler view - problem with queue percent rounding

2016-06-10 Thread Akhil PB
) - contrib/views/capacity-scheduler/src/main/resources/ui/app/components/totalCapacity.js c36839c contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js 4aeb442 Diff: https://reviews.apache.org/r/48398/diff/ Testing --- Thanks, Akhil PB

Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-09 Thread Akhil PB
contrib/views/capacity-scheduler/src/main/resources/ui/app/views/queuesconf.js 5cef8e5 Diff: https://reviews.apache.org/r/48487/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Review Request 47814: Capacity Scheduler View - Save only, Save and Restart Rm and Delete queue integrations

2016-05-25 Thread Akhil PB
/editQueueCapacity.js 5c3793f contrib/views/capacity-scheduler/src/main/resources/ui/app/views/queuesconf.js 7cb71a7 Diff: https://reviews.apache.org/r/47814/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 47643: Capacity Scheduler View - Edit and validate queue name

2016-05-22 Thread Akhil PB
://reviews.apache.org/r/47643/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Review Request 47643: Capacity Scheduler View - Edit and validate queue name

2016-05-20 Thread Akhil PB
/resources/ui/app/templates/capsched/queuesconf.hbs d57b2ba contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched/queuesconf/editqueue.hbs c5c910a Diff: https://reviews.apache.org/r/47643/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 46770: Capacity Scheduler View: New UI - queues save integration

2016-05-09 Thread Akhil PB
/editQueueCapacity.js 2628553 Diff: https://reviews.apache.org/r/46770/diff/ Testing --- Manual Unit Testing Thanks, Akhil PB

Re: Review Request 44704: CapSched View: Revamping the old UI

2016-04-21 Thread Akhil PB
sit: 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 re

Re: Review Request 44704: CapSched View: Revamping the old UI

2016-04-13 Thread Akhil PB
/stackoverflow.com/questions/11628489/emberjs-how-to-mark-active-menu-item-using-router-infrastructure - Akhil --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review1

Re: Review Request 44704: CapSched View: Revamping the old UI

2016-04-13 Thread Akhil PB
/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