----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45012/#review124221 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py (line 694) <https://reviews.apache.org/r/45012/#comment186682> do you want to inline this since propertyvalue is used only once. ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py (line 979) <https://reviews.apache.org/r/45012/#comment186681> Do you want to set (constant) variable for a magic number 4? ``` REPLACE_DATANODE_ON_FAILURE_THREASHOLD = 4 # my poor naming skill... if numSegments > REPLACE_DATANODE_ON_FAILURE_THREASHOLD and value ... : ... elif numSegments <= REPLACE_DATANODE_ON_FAILURE_THREASHOLD and value ... : ... ``` and you can use it on line 694 - jun aoki On March 18, 2016, 2:01 a.m., Alexander Denissov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45012/ > ----------------------------------------------------------- > > (Updated March 18, 2016, 2:01 a.m.) > > > Review request for Ambari, Alejandro Fernandez, bhuvnesh chaudhary, Goutam > Tadi, jun aoki, Jayush Luniya, Lav Jain, and Sumit Mohanty. > > > Bugs: AMBARI-15499 > https://issues.apache.org/jira/browse/AMBARI-15499 > > > Repository: ambari > > > Description > ------- > > Recommendation and validation logic for HAWQ hdfs-client.xml > output.replace-datanode-on-failure property > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/configuration/hdfs-client.xml > c2a756a > ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py > c0bc4e2 > ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py > 4f0b4b7 > > Diff: https://reviews.apache.org/r/45012/diff/ > > > Testing > ------- > > manual > > ---------------------------------------------------------------------- > Ran 253 tests in 7.111s > > OK > ---------------------------------------------------------------------- > Total run:927 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alexander Denissov > >
