----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39530/#review103507 -----------------------------------------------------------
ambari-server/src/main/resources/stacks/HDP/2.3/services/SPARK/configuration/spark-thrift-cmd-opts-properties.xml (line 21) <https://reviews.apache.org/r/39530/#comment161555> What we meant is that you can introduce a property spark-thrift-cmd-opts in spark-env.xml and have it contain the value user wants to supply. Then in the scripts you can refer to it as spark_thrift_cmd_opts_properties = str(config['configurations']['spark-env']['spark-thrift-cmd-opts']) - Sumit Mohanty On Oct. 21, 2015, 10:33 p.m., Judy Nash wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39530/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2015, 10:33 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. > > > Bugs: AMBARI-13513 > https://issues.apache.org/jira/browse/AMBARI-13513 > > > Repository: ambari > > > Description > ------- > > Ambari-13513:[trunk][2.1.2] Add cmd option config to spark thrift server > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/params.py > 518ba6d > > ambari-server/src/main/resources/common-services/SPARK/1.2.0.2.2/package/scripts/spark_service.py > 68a395b > > ambari-server/src/main/resources/stacks/HDP/2.3/services/SPARK/configuration/spark-thrift-cmd-opts-properties.xml > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.3/services/SPARK/metainfo.xml > 14161b4 > ambari-server/src/test/python/stacks/2.3/SPARK/test_spark_thrift_server.py > b3b8235 > ambari-server/src/test/python/stacks/2.3/configs/spark_default.json 9f3fb90 > > Diff: https://reviews.apache.org/r/39530/diff/ > > > Testing > ------- > > 1. Updated Unit Tests > 2. E2E test on HDInsight cluster > > > Thanks, > > Judy Nash > >