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



ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/configuration/ranger-admin-site.xml
 (line 371)
<https://reviews.apache.org/r/38916/#comment159272>

    Instead of using a macro, The logic should be made part of stack advisor 
and we should keep using absolute value for the property for following reasons:
    * we can gracefully handle upgrade scenario.
    * We can use Service config version feature on ambari-web to compare it's 
actual absolute value with previous ones when needed


- Jaimin Jetly


On Oct. 1, 2015, 6:50 a.m., Gautam Borad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38916/
> -----------------------------------------------------------
> 
> (Updated Oct. 1, 2015, 6:50 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, Mahadev Konar, 
> Sumit Mohanty, Selvamohan Neethiraj, Velmurugan Periasamy, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-13273
>     https://issues.apache.org/jira/browse/AMBARI-13273
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Patch contains changes on ambari stacks to allow Ranger Usersync to be 
> installed on separate host.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py
>  7c69e57 
>   
> ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/service_check.py
>  85ac6f5 
>   
> ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/setup_ranger_xml.py
>  77e487f 
>   
> ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/configuration/ranger-admin-site.xml
>  7423e8d 
> 
> Diff: https://reviews.apache.org/r/38916/diff/
> 
> 
> Testing
> -------
> 
> Unit test report:
> ----------------------------------------------------------------------
> Ran 264 tests in 8.125s
> 
> OK
> ----------------------------------------------------------------------
> Total run:838
> Total errors:0
> Total failures:0
> OK
> 
> Process finished with exit code 0
> 
> 
> Thanks,
> 
> Gautam Borad
> 
>

Reply via email to