> On Aug. 31, 2015, 10:16 p.m., Rohini Palaniswamy wrote: > > sharelib/spark/pom.xml, line 137 > > <https://reviews.apache.org/r/37452/diff/1-2/?file=1039663#file1039663line137> > > > > Are we removing spark-core altogether as other dependencies will bring > > it?
I think the previous version of the patch was moving the spark-core dependency definition in the pom file, but now it's not, so RB is showing it funny like this. If you look at the patch on the JIRA, it's not removing spark-core. In any case, we don't want to remove it and I'll make sure it's not when committing. - Robert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37452/#review97166 ----------------------------------------------------------- On Aug. 28, 2015, 5:03 a.m., Robert Kanter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37452/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2015, 5:03 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-2277 > https://issues.apache.org/jira/browse/OOZIE-2277 > > > Repository: oozie-git > > > Description > ------- > > https://issues.apache.org/jira/browse/OOZIE-2277 > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/service/SparkConfigurationService.java > 1b7cf4a > core/src/main/resources/oozie-default.xml 32a1df0 > > core/src/test/java/org/apache/oozie/service/TestSparkConfigurationService.java > b2c499d > docs/src/site/twiki/DG_SparkActionExtension.twiki 32ebe12 > sharelib/spark/pom.xml 6f7e74a > sharelib/spark/src/main/java/org.apache.oozie.action.hadoop/SparkMain.java > b18a0b9 > > sharelib/spark/src/test/java/org/apache/oozie/action/hadoop/TestSparkActionExecutor.java > f271abc > > Diff: https://reviews.apache.org/r/37452/diff/ > > > Testing > ------- > > - Ran unit tests with Hadoop 1 and Hadoop 2 > - Ran in a Hadoop 2 cluster with local, yarn-client, and yarn-cluster modes > > > Thanks, > > Robert Kanter > >
