----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30679/#review72000 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On Feb. 10, 2015, 6:19 p.m., Yurii Shylov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30679/ > ----------------------------------------------------------- > > (Updated Feb. 10, 2015, 6:19 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate > Cole. > > > Bugs: AMBARI-9498 > https://issues.apache.org/jira/browse/AMBARI-9498 > > > Repository: ambari > > > Description > ------- > > The hosts master check is not properly checking all hosts. The check is > currently checking only things like NN master etc. This needs to check any > master that is part of the upgrade-2.2.xml upgrade script, including any > service-defined master (like Hive masters, ZK, etc). > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/checks/AbstractCheckDescriptor.java > 647f2d1 > > ambari-server/src/main/java/org/apache/ambari/server/checks/HostsMasterMaintenanceCheck.java > 91ca65a > > ambari-server/src/test/java/org/apache/ambari/server/state/CheckHelperTest.java > c1ea6e4 > > Diff: https://reviews.apache.org/r/30679/diff/ > > > Testing > ------- > > In progress > > > Thanks, > > Yurii Shylov > >
