----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68317/#review207151 -----------------------------------------------------------
sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopMain.java Lines 32 (patched) <https://reviews.apache.org/r/68317/#comment290433> We could use https://github.com/apache/hadoop/blob/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java#L289 here sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopMain.java Lines 33 (patched) <https://reviews.apache.org/r/68317/#comment290432> Please use the constant from LauncherAM: https://github.com/apache/oozie/blob/master/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAM.java#L90 sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopMain.java Lines 66-68 (patched) <https://reviews.apache.org/r/68317/#comment290431> Please put this into a finally or @after block so the cleanup happens even if the test fails - Peter Cseh On Aug. 13, 2018, 8:20 a.m., Denes Bodo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68317/ > ----------------------------------------------------------- > > (Updated Aug. 13, 2018, 8:20 a.m.) > > > Review request for oozie and AndrĂ¡s Piros. > > > 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 > 4a41941 > > sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMain.java > e6e182c > > 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 > 27f9306 > > sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopMain.java > d6f96d5 > > > Diff: https://reviews.apache.org/r/68317/diff/1/ > > > Testing > ------- > > > Thanks, > > Denes Bodo > >