----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41220/#review109851 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/params.py (line 156) <https://reviews.apache.org/r/41220/#comment169507> This will work for a fresh install of 2.3.0.0 and 2.3.4.0. However, in an RU from 2.2 to 2.3.4.0 and then downgrade, Script.is_hdp_stack_greater_or_equal("2.3") == true and the config 'spark-thrift-sparkconf' exists, so the downgrade back to 2.2 will be using 2.3.4.0 configs. ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/params.py (line 163) <https://reviews.apache.org/r/41220/#comment169505> Are these properties always available in HDP 2.3, e.g., 2.3.0.0 and 2.3.2.0? - Alejandro Fernandez On Dec. 10, 2015, 8:03 p.m., Oliver Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41220/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2015, 8:03 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Sumit Mohanty, and Sebastian > Toader. > > > Bugs: AMBARI-14330 > https://issues.apache.org/jira/browse/AMBARI-14330 > > > Repository: ambari > > > Description > ------- > > In this patch: https://reviews.apache.org/r/41133/ > Although the issue was fixed, "commandParams/version" should not be > redefined, because it used by RU/EU. Instead of using that parameter I using > to less detailed version (2 digit) and an another condition (spark > thriftserver config exists) to adding the needed configuration properties. > (also I reverted the removed comment) > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params_linux.py > c327c42 > > ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/params.py > 732f32c > > Diff: https://reviews.apache.org/r/41220/diff/ > > > Testing > ------- > > Python tests succeeded for ambari-server. > Other Unit tests are in progress. > > > Thanks, > > Oliver Szabo > >
