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



ambari-server/src/main/java/org/apache/ambari/server/agent/DiskInfo.java
<https://reviews.apache.org/r/32784/#comment128157>

    I do not see any usages of this property in this patch. How is it used?



ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/configuration/hdfs-site.xml
<https://reviews.apache.org/r/32784/#comment128158>

    Is this necessary?


- Srimanth Gunturi


On April 2, 2015, 6:20 p.m., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32784/
> -----------------------------------------------------------
> 
> (Updated April 2, 2015, 6:20 p.m.)
> 
> 
> Review request for Ambari, Myroslav Papirkovskyy and Srimanth Gunturi.
> 
> 
> Bugs: AMBARI-10332
>     https://issues.apache.org/jira/browse/AMBARI-10332
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The stack-advisor /recommendations end-point needs to be improved to support 
> enhanced-configs requirements for HDFS. This includes the support for 
> property_value_attributes.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/agent/DiskInfo.java 
> e3fb88e 
>   
> ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommand.java
>  ce67002 
>   
> ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/configuration/hdfs-site.xml
>  9c704f9 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py 
> 2405954 
>   ambari-server/src/main/resources/stacks/HDP/2.2/services/stack_advisor.py 
> 87748d2 
>   ambari-server/src/test/python/stacks/2.2/common/test_stack_advisor.py 
> d12c067 
> 
> Diff: https://reviews.apache.org/r/32784/diff/
> 
> 
> Testing
> -------
> 
> unit tests passed
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>

Reply via email to