> On March 21, 2017, 11:05 p.m., Robert Kanter wrote: > > core/src/main/resources/oozie-default.xml > > Lines 2893-2903 (original) > > <https://reviews.apache.org/r/57680/diff/2/?file=1670799#file1670799line2893> > > > > Shouldn't we leave this here for now?
I don't know how log we should keep this. The blacklist property already serves the same purpose and the code is in place to handle backwards compatiblity. We can keep it here until the next release or so. - Peter ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57680/#review169633 ----------------------------------------------------------- On March 21, 2017, 1:50 p.m., Peter Cseh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57680/ > ----------------------------------------------------------- > > (Updated March 21, 2017, 1:50 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-2812 > https://issues.apache.org/jira/browse/OOZIE-2812 > > > Repository: oozie-git > > > Description > ------- > > OOZIE-2812 > SparkConfigurationService should support loading configurations from multiple > Spark versions > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/action/hadoop/SparkActionExecutor.java > 1a3197abb52f80b545dcc8634a8cac7a281a9eac > core/src/main/java/org/apache/oozie/service/SparkConfigurationService.java > b15cce0d6813375778abf1d2bbe09e83734d19f0 > > core/src/main/java/org/apache/oozie/service/sparkconfiguration/SparkConfigReader.java > PRE-CREATION > > core/src/main/java/org/apache/oozie/service/sparkconfiguration/SparkConfigurationStorage.java > PRE-CREATION > core/src/main/resources/oozie-default.xml > b481887122be0112c763ad667d1de18747d1ad9b > > core/src/test/java/org/apache/oozie/service/TestSparkConfigurationService.java > 0e00a457890d1f4b8e86c19043cd597928cdd9bb > > core/src/test/java/org/apache/oozie/service/sparkconfiguration/TestSparkConfigurationStorage.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/57680/diff/2/ > > > Testing > ------- > > > Thanks, > > Peter Cseh > >
