----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54970/#review160089 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py (line 70) <https://reviews.apache.org/r/54970/#comment231082> I don't think spark_home is being used by Oozie Python script. Can this be shorten to spark_conf = format("{stack_root}/current/spark-client/conf") - Di Li On Dec. 23, 2016, 11:58 a.m., ping wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54970/ > ----------------------------------------------------------- > > (Updated Dec. 23, 2016, 11:58 a.m.) > > > Review request for Ambari, Alejandro Fernandez and Di Li. > > > Bugs: AMBARI-19250 > https://issues.apache.org/jira/browse/AMBARI-19250 > > > Repository: ambari > > > Description > ------- > > Change default Spark configuration property in Oozie configuration settings > to valid value > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py > a2d725c > > ambari-server/src/main/resources/common-services/OOZIE/4.2.0.2.5/configuration/oozie-site.xml > cdb6972 > > Diff: https://reviews.apache.org/r/54970/diff/ > > > Testing > ------- > > I have tested the fix by: > 1) copy the changed files into ambari host and restart ambari-server > 2) delete oozie service and then add it back > 3) check the property value is point to the spark configuration directory > correctly > > > Thanks, > > ping wang > >
