----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53066/#review153801 -----------------------------------------------------------
Ship it! ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListener.java (line 146) <https://reviews.apache.org/r/53066/#comment223261> Some JavaDoc here. - Jonathan Hurley On Oct. 25, 2016, 11:38 a.m., Dmitro Lisnichenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53066/ > ----------------------------------------------------------- > > (Updated Oct. 25, 2016, 11:38 a.m.) > > > Review request for Ambari, Jonathan Hurley and Nate Cole. > > > Bugs: AMBARI-18651 > https://issues.apache.org/jira/browse/AMBARI-18651 > > > Repository: ambari > > > Description > ------- > > Previous to HDP-2.5.0.0, ZKFC was not included with hdp-select. Therefore, > the metainfo for ZKFC used {{<versionAdvertised>false</versionAdvertised>}}. > It was recently discovered that 2.5.0.0-1154+ has added this capability. > > * Change versionAdvertised > * Add code to ZKFC python to hdp-select. This code MUST check the > stack_feature to ensure that it is executed ONLY for HDP-2.5+ > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/constants.py > 1396bd8 > > ambari-server/src/main/java/org/apache/ambari/server/events/AmbariEvent.java > 912c441 > > ambari-server/src/main/java/org/apache/ambari/server/events/StackUpgradeFinishEvent.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/StackUpgradeFinishListener.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListener.java > 87247eb > > ambari-server/src/main/java/org/apache/ambari/server/events/publishers/VersionEventPublisher.java > 5b32c4e > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java > a07d0e6 > > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponent.java > 983cbdf > > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentImpl.java > 3e805a0 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/zkfc_slave.py > aa0ab0f > > ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json > dbde58b > ambari-server/src/main/resources/stacks/HDP/2.5/services/HDFS/metainfo.xml > a3e4a64 > > ambari-server/src/test/java/org/apache/ambari/server/audit/request/creator/RepositoryEventCreatorTest.java > 3479334 > > ambari-server/src/test/java/org/apache/ambari/server/audit/request/creator/RepositoryVersionEventCreatorTest.java > 18e2d3f > > ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/StackUpgradeFinishListenerTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListenerTest.java > d22622e > > Diff: https://reviews.apache.org/r/53066/diff/ > > > Testing > ------- > > few runs on live cluster > > mvn clean test > > > Thanks, > > Dmitro Lisnichenko > >