You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Sreenath Somarajapuram <ss...@hortonworks.com> on 2016/02/09 10:13:20 UTC

Re: Review Request 43009: View: Capacity Scheduler View - Provide dropdown for scheduling calculator

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




contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js (line 421)
<https://reviews.apache.org/r/43009/#comment179608>

    Its more redable to split at ,


- Sreenath Somarajapuram


On Jan. 30, 2016, 7:05 a.m., Akhil PB wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43009/
> -----------------------------------------------------------
> 
> (Updated Jan. 30, 2016, 7:05 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Pallav Kulshreshtha, and Sreenath Somarajapuram.
> 
> 
> Bugs: AMBARI-14852
>     https://issues.apache.org/jira/browse/AMBARI-14852
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Implement dropdown for scheduling calculator in capacity scheduler view.
> 
> 
> Diffs
> -----
> 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js 416ff43 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less c0acf4f 
>   contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/schedulerPanel.hbs 4419aa0 
> 
> Diff: https://reviews.apache.org/r/43009/diff/
> 
> 
> Testing
> -------
> 
> Done some manual unit testing. The dropdown will be only shown in ambari operator mode.
> 
> 
> Thanks,
> 
> Akhil PB
> 
>