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

2016-04-27 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review130753 --- Ship it! Ship It! - Sreenath Somarajapuram On April 27, 2016

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

2016-04-27 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/ --- (Updated April 27, 2016, 9:45 a.m.) Review request for Ambari and Sreenath Soma

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

2016-04-27 Thread Akhil PB
> On April 4, 2016, 9:05 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched.hbs, > > line 28 > > > > > > Whay are we having an anchor tags inside lin

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

2016-04-27 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review130740 --- contrib/views/capacity-scheduler/src/main/resources/ui/app/contro

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

2016-04-26 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/ --- (Updated April 26, 2016, 10:43 a.m.) Review request for Ambari and Sreenath Som

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

2016-04-21 Thread Akhil PB
> 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 > > > > > > Can use bunded conditional c

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

2016-04-13 Thread Akhil PB
> On April 4, 2016, 9:05 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/helpers/queueMappingParser.js, > > line 19 > > > > > > Looks like you can delete this file.

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

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

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

2016-04-04 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review126813 --- contrib/views/capacity-scheduler/src/main/resources/ui/app/contro

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

2016-04-04 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review126808 --- contrib/views/capacity-scheduler/src/main/resources/ui/app/templa

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

2016-04-04 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44704/#review126795 --- contrib/views/capacity-scheduler/src/main/resources/ui/app/helper