----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68317/#review212890 -----------------------------------------------------------
sharelib/sqoop/src/main/java/org/apache/oozie/action/hadoop/SqoopMain.java Lines 45 (patched) <https://reviews.apache.org/r/68317/#comment298776> Could we use LauncherAM.OOZIE_ACTION_CONF_XML instead? sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopMain.java Lines 62-64 (patched) <https://reviews.apache.org/r/68317/#comment298775> Could use AssertNotNull - Andras Salamon On Feb. 15, 2019, 11:33 a.m., Denes Bodo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68317/ > ----------------------------------------------------------- > > (Updated Feb. 15, 2019, 11:33 a.m.) > > > Review request for oozie and Andras Salamon. > > > Bugs: OOZIE-3326 > https://issues.apache.org/jira/browse/OOZIE-3326 > > > Repository: oozie-git > > > Description > ------- > > SqoopMain needs to support tez delegation tokens for hive-imports. > Implementation is similar to that of HiveMain and Hive2Main. > > At present, hive-import will fail to start a tez session in secure > environment. > > > Diffs > ----- > > sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAM.java > 63afd91d3 > > sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMain.java > b6599f7f3 > > sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/SystemEnvironment.java > PRE-CREATION > sharelib/sqoop/src/main/java/org/apache/oozie/action/hadoop/SqoopMain.java > 27f9306a0 > > sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopMain.java > d6f96d546 > > > Diff: https://reviews.apache.org/r/68317/diff/5/ > > > Testing > ------- > > > Thanks, > > Denes Bodo > >
