----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39237/#review102274 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java (line 1480) <https://reviews.apache.org/r/39237/#comment159852> This is not valid for fs.DefaultFS that is not hdfs. So we can add this property only if the fs is hdfs. E.g. it will not work for wasb:// or ecs:// etc. Also, lets add it only if the property does not exist in hdfs-site. - Sumit Mohanty On Oct. 12, 2015, 4:36 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39237/ > ----------------------------------------------------------- > > (Updated Oct. 12, 2015, 4:36 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Myroslav Papirkovskyy, and > Vitalyi Brodetskyi. > > > Bugs: AMBARI-13373 > https://issues.apache.org/jira/browse/AMBARI-13373 > > > Repository: ambari > > > Description > ------- > > Ran hdfs balancer via ambari on a cluster that had HA enabled and it failed. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/AbstractUpgradeCatalog.java > 62cd868 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java > 12a1f44 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog213.java > dcdbb85 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog213Test.java > a54dee2 > > Diff: https://reviews.apache.org/r/39237/diff/ > > > Testing > ------- > > Unit tests passed > > > Thanks, > > Dmytro Sen > >
