Re: Review Request 48569: On Enabling HS Interactive default queue shown for llap is 'default' queue

2016-06-10 Thread Zhe (Joe) Wang

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


Ship it!




Ship It!

- Zhe (Joe) Wang


On June 11, 2016, 12:07 a.m., Jaimin Jetly wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48569/
> ---
> 
> (Updated June 11, 2016, 12:07 a.m.)
> 
> 
> Review request for Ambari, Zhe (Joe) Wang, Srimanth Gunturi, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-17175
> https://issues.apache.org/jira/browse/AMBARI-17175
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> 1. Deploy ambari secure cluster
> 2. Enable HSI
> Expected: llap queue is created by ambari and is set as default for llap
> Actual: ldap queue is not created and default queue is set for ldap
> 
> 
> Diffs
> -
> 
>   ambari-web/app/views/common/configs/widgets/combo_config_widget_view.js 
> 0bf05f6 
> 
> Diff: https://reviews.apache.org/r/48569/diff/
> 
> 
> Testing
> ---
> 
> Verified that the patch fixes the issue on a cluster
> ambari-web unit tests passes successfully with the patch:
> 28668 tests complete (28 seconds)
> 154 tests pending
> 
> 
> Thanks,
> 
> Jaimin Jetly
> 
>



Review Request 48569: On Enabling HS Interactive default queue shown for llap is 'default' queue

2016-06-10 Thread Jaimin Jetly

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

Review request for Ambari, Zhe (Joe) Wang, Srimanth Gunturi, and Yusaku Sako.


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


Repository: ambari


Description
---

1. Deploy ambari secure cluster
2. Enable HSI
Expected: llap queue is created by ambari and is set as default for llap
Actual: ldap queue is not created and default queue is set for ldap


Diffs
-

  ambari-web/app/views/common/configs/widgets/combo_config_widget_view.js 
0bf05f6 

Diff: https://reviews.apache.org/r/48569/diff/


Testing
---

Verified that the patch fixes the issue on a cluster
ambari-web unit tests passes successfully with the patch:
28668 tests complete (28 seconds)
154 tests pending


Thanks,

Jaimin Jetly