> On Окт. 12, 2015, 5:07 п.п., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/AbstractUpgradeCatalog.java, > > line 333 > > <https://reviews.apache.org/r/39237/diff/1/?file=1096068#file1096068line333> > > > > Should we check both conditions? > > dfs.ha.automatic-failover.enabled and dfs.ha.namenodes.*?
In check if HA is enabled removed explicit check for dfs.ha.automatic-failover.enabled=true. - Dmytro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39237/#review102277 ----------------------------------------------------------- On Окт. 13, 2015, 3:53 п.п., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39237/ > ----------------------------------------------------------- > > (Updated Окт. 13, 2015, 3:53 п.п.) > > > 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/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 > >
