----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67834/#review206596 -----------------------------------------------------------
tools/src/test/java/org/apache/oozie/tools/IntegrationTestOozieSharelibCLI.java Lines 46 (patched) <https://reviews.apache.org/r/67834/#comment289604> Shouldn't it be rather `@Rule public TemporaryFolder tmpFolder = new TemporaryFolder()`? tools/src/test/java/org/apache/oozie/tools/IntegrationTestOozieSharelibCLI.java Lines 82 (patched) <https://reviews.apache.org/r/67834/#comment289605> Please provide more context to the assertion error message. tools/src/test/java/org/apache/oozie/tools/IntegrationTestOozieSharelibCLI.java Lines 102 (patched) <https://reviews.apache.org/r/67834/#comment289606> Please provide more context to the assertion error message. tools/src/test/java/org/apache/oozie/tools/IntegrationTestOozieSharelibCLI.java Lines 148 (patched) <https://reviews.apache.org/r/67834/#comment289607> Please provide more context to the assertion error message. tools/src/test/java/org/apache/oozie/tools/TestOozieSharelibCLI.java Lines 81-82 (original), 80-81 (patched) <https://reviews.apache.org/r/67834/#comment289608> Please provide more context to the assertion error message. tools/src/test/java/org/apache/oozie/tools/TestOozieSharelibCLI.java Line 169 (original), 103 (patched) <https://reviews.apache.org/r/67834/#comment289609> Please provide more context to the assertion error message. - András Piros On July 25, 2018, 8:48 a.m., Kinga Marton wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67834/ > ----------------------------------------------------------- > > (Updated July 25, 2018, 8:48 a.m.) > > > Review request for oozie, András Piros, Peter Cseh, and Peter Bacsko. > > > Repository: oozie-git > > > Description > ------- > > Right now sharelib can be created via sharelib create -fs FS_URI -locallib > SHARED_LIBRARY where the SHARED_LIBRARY can be a tarbal or a folder. > It would be nice to have the possibility to define additional folders to be > uploaded into the sharelib, so the users don't have to copy or link the files > together on their machine. > The syntax could be something like -additional-lib > sharelibName=/path/to/source/;/path/to/some/file,sharelibName2=/path/to/some/folder > > > Diffs > ----- > > docs/src/site/twiki/AG_Install.twiki 46363a3b3 > tools/src/main/java/org/apache/oozie/tools/OozieSharelibCLI.java 75e932c02 > > tools/src/test/java/org/apache/oozie/tools/IntegrationTestOozieSharelibCLI.java > PRE-CREATION > tools/src/test/java/org/apache/oozie/tools/OozieSharelibFileOperations.java > d344300ed > tools/src/test/java/org/apache/oozie/tools/TestConcurrentCopyFromLocal.java > d77eba69d > tools/src/test/java/org/apache/oozie/tools/TestCopyTaskCallable.java > bce0433c6 > tools/src/test/java/org/apache/oozie/tools/TestOozieSharelibCLI.java > 5929e5ca0 > > tools/src/test/java/org/apache/oozie/tools/TestOozieSharelibCLIExtraArgsParser.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/67834/diff/7/ > > > Testing > ------- > > Tested manually + added integration test > > > Thanks, > > Kinga Marton > >
