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

2017-02-10 Thread Nitiraj Rathore

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


Ship it!




Ship It!

- Nitiraj Rathore


On Feb. 10, 2017, 7:38 a.m., Akhil PB wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56379/
> ---
> 
> (Updated Feb. 10, 2017, 7:38 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav 
> Kulshreshtha, and Sreenath Somarajapuram.
> 
> 
> Bugs: AMBARI-19418
> https://issues.apache.org/jira/browse/AMBARI-19418
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Feature to support queue priority config settings in YARN
> 
> 
> Diffs
> -
> 
>   
> contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js
>  589dcd0 
>   
> contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js
>  2f9f2fe 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/models/queue.js 
> 9198aa5 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 
> 940d0f8 
>   contrib/views/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
> 
>



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

2017-02-10 Thread Sreenath Somarajapuram

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


Ship it!




Ship It!

- Sreenath Somarajapuram


On Feb. 10, 2017, 7:38 a.m., Akhil PB wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56379/
> ---
> 
> (Updated Feb. 10, 2017, 7:38 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav 
> Kulshreshtha, and Sreenath Somarajapuram.
> 
> 
> Bugs: AMBARI-19418
> https://issues.apache.org/jira/browse/AMBARI-19418
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Feature to support queue priority config settings in YARN
> 
> 
> Diffs
> -
> 
>   
> contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js
>  589dcd0 
>   
> contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js
>  2f9f2fe 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/models/queue.js 
> 9198aa5 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 
> 940d0f8 
>   contrib/views/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
> 
>



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

2017-02-10 Thread Gaurav Nagar

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


Ship it!




Ship It!

- Gaurav Nagar


On Feb. 10, 2017, 7:38 a.m., Akhil PB wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56379/
> ---
> 
> (Updated Feb. 10, 2017, 7:38 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav 
> Kulshreshtha, and Sreenath Somarajapuram.
> 
> 
> Bugs: AMBARI-19418
> https://issues.apache.org/jira/browse/AMBARI-19418
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Feature to support queue priority config settings in YARN
> 
> 
> Diffs
> -
> 
>   
> contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js
>  589dcd0 
>   
> contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js
>  2f9f2fe 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/models/queue.js 
> 9198aa5 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 
> 940d0f8 
>   contrib/views/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
> 
>



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

2017-02-09 Thread Akhil PB

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

(Updated Feb. 10, 2017, 7:38 a.m.)


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


Changes
---

Patch updated


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


Repository: ambari


Description
---

Feature to support queue priority config settings in YARN


Diffs (updated)
-

  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js 
589dcd0 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js
 2f9f2fe 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/models/queue.js 
9198aa5 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 
940d0f8 
  contrib/views/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 56379: AMBARI-19418: Support setup queue priority in Ambari - Capacity scheduler view

2017-02-07 Thread Akhil PB

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

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


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


Repository: ambari


Description
---

Feature to support queue priority config settings in YARN


Diffs
-

  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queue.js 
589dcd0 
  
contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js
 2f9f2fe 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/models/queue.js 
9198aa5 
  contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js 
940d0f8 
  contrib/views/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