----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64388/#review193122 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java Lines 3092-3096 (patched) <https://reviews.apache.org/r/64388/#comment271624> One more question - what is the reason for this code? Is it necessary for this fix? - Jonathan Hurley On Dec. 7, 2017, 9:43 a.m., Myroslav Papirkovskyy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64388/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2017, 9:43 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, and Sid Wagle. > > > Bugs: AMBARI-22605 > https://issues.apache.org/jira/browse/AMBARI-22605 > > > Repository: ambari > > > Description > ------- > > STR: > Deploy minimal cluster (HDFS, YARN+MR2, ZooKeeper). > Add Tez service. > Service check hangs for a long time. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 85e0c1cf130974377abb7b685f191e20324b090e > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListener.java > d5a1ac4fd06377a992a8286d6fca366536a25157 > > > Diff: https://reviews.apache.org/r/64388/diff/2/ > > > Testing > ------- > > mvn clean test > manual check > > > Thanks, > > Myroslav Papirkovskyy > >
