----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36508/#review91743 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java (lines 237 - 241) <https://reviews.apache.org/r/36508/#comment145360> ConfigureAction is not the place for this change. It should be in UpgradeResourceProvider.processConfigurations(), where we make new configs from the stack values. ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java (lines 449 - 459) <https://reviews.apache.org/r/36508/#comment145361> Should not be done here. ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (lines 681 - 691) <https://reviews.apache.org/r/36508/#comment145364> You're changing the behavior of this method; should have a test. - Nate Cole On July 15, 2015, 10:36 a.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36508/ > ----------------------------------------------------------- > > (Updated July 15, 2015, 10:36 a.m.) > > > Review request for Ambari, Jonathan Hurley and Nate Cole. > > > Bugs: AMBARI-12419 > https://issues.apache.org/jira/browse/AMBARI-12419 > > > Repository: ambari > > > Description > ------- > > The issue is noticed in an RU cluster where rolling upgrade has to be > performed with ambari2.1.0. After installing dal bits we are setting final > flag for a property 'tez.am.resource.memory.mb' to final which you can see in > V3 config of the cluster. Then enabling security and HA is performed. After > these operations we see that the final flag was turned off. Upon checking the > versions, V4 doesn't have this final flag and changes between V3 and V4 is > that, security is enabled at V4. (HiveServer2 Authentication is set to > kerberos in V4 not in V3). > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java > c717582 > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java > 8b4b1e2 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog170.java > c54039f > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog200.java > 2c9bf30 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog170Test.java > 93f7f8c > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog200Test.java > 484c18d > > Diff: https://reviews.apache.org/r/36508/diff/ > > > Testing > ------- > > unit tests are in progress > > > Thanks, > > Dmytro Sen > >
