----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62470/#review187014 -----------------------------------------------------------
core/src/main/java/org/apache/oozie/action/hadoop/SparkActionExecutor.java Lines 149-151 (original) <https://reviews.apache.org/r/62470/#comment263966> I missed this earlier in the review. If we have this, then we should set the env here (will require method signature change) and remove the earlier code for setting SPARK_HOME that was done in setupLauncherConf as well. - Rohini Palaniswamy On Oct. 3, 2017, 9:52 p.m., Satish Saley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62470/ > ----------------------------------------------------------- > > (Updated Oct. 3, 2017, 9:52 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-3062 > https://issues.apache.org/jira/browse/OOZIE-3062 > > > Repository: oozie-git > > > Description > ------- > > OOZIE-3062 Set HADOOP_CONF_DIR for spark action > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > 76d0daa4 > core/src/main/java/org/apache/oozie/action/hadoop/SparkActionExecutor.java > 80d64ec8 > > sharelib/spark/src/test/java/org/apache/oozie/action/hadoop/TestSparkActionExecutor.java > d97f1f06 > > > Diff: https://reviews.apache.org/r/62470/diff/4/ > > > Testing > ------- > > > Thanks, > > Satish Saley > >
