> On July 31, 2016, 10:02 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py, > > line 129 > > <https://reviews.apache.org/r/50594/diff/2/?file=1458673#file1458673line129> > > > > I think the previous code was safer - spark_javaopts_properties = > > default("/configurations/spark-javaopts-properties/content", " ") > > > > Any reason to switch to if ... else ...
I see, I will revert it back. - Saisai ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50594/#review144288 ----------------------------------------------------------- On July 31, 2016, 3:44 a.m., Saisai Shao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50594/ > ----------------------------------------------------------- > > (Updated July 31, 2016, 3:44 a.m.) > > > Review request for Ambari, Jayush Luniya and Sumit Mohanty. > > > Bugs: AMBARI-17954 > https://issues.apache.org/jira/browse/AMBARI-17954 > > > Repository: ambari > > > Description > ------- > > Fix Spark hdp.version issues in upgrading and fresh install > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/constants.py > 42ecf6a > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java > 5495655 > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/configuration/spark-defaults.xml > 646239e > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py > 86aeb70 > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/setup_spark.py > 2b5d36b > > ambari-server/src/main/resources/common-services/SPARK/1.5.2/configuration/spark-thrift-sparkconf.xml > e27c986 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json > 6d26052 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/config-upgrade.xml > b295cc9 > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.4.xml > 5b20a72 > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.5.xml > 5d2cc54 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.4.xml > c9f9823 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml > d0332ed > > ambari-server/src/main/resources/stacks/HDP/2.4/services/SPARK/configuration/spark-defaults.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.4/services/SPARK/configuration/spark-thrift-sparkconf.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.5/services/SPARK/configuration/spark-defaults.xml > b3d05d8 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog240Test.java > 34ca199 > ambari-server/src/test/python/stacks/2.2/SPARK/test_job_history_server.py > b2ed5df > ambari-server/src/test/python/stacks/2.2/SPARK/test_spark_client.py ed331ea > ambari-server/src/test/python/stacks/2.3/SPARK/test_spark_thrift_server.py > a1abdfa > > Diff: https://reviews.apache.org/r/50594/diff/ > > > Testing > ------- > > Manual test with different scenarios: > > 1. Fresh install of HDP 2.3.6, 2.4.3, 2.5.0 > 2. Upgrade for 2.3.6 to 2.5.0. > 3. Downgrade from 2.5.0 to 2.3.6. > > > Thanks, > > Saisai Shao > >
