----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13931/#review25911 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java <https://reviews.apache.org/r/13931/#comment50559> addToCache does lot of checks which is not required. Why not directly add to DistributedCache and create symlink? trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java <https://reviews.apache.org/r/13931/#comment50564> tmpShareLibPath trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java <https://reviews.apache.org/r/13931/#comment50567> copyLauncherJarsToShareLib(fs, tmpShareLibPath) trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java <https://reviews.apache.org/r/13931/#comment50563> Don't need this line trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java <https://reviews.apache.org/r/13931/#comment50570> (1000 * 60 * 60 * 24 * conf.getInt(LAUNCHERJAR_LIB_RETENTION, 7))) outside the loop trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java <https://reviews.apache.org/r/13931/#comment50572> Nitpick. Comment on the English Language (Not java language :) ) "Deleted". Capitalize first letter in the sentence. trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java <https://reviews.apache.org/r/13931/#comment50573> clear the maps in destroy trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java <https://reviews.apache.org/r/13931/#comment50578> as maven has target/classes in classpath and not the jar because test phase is before package phase - Rohini Palaniswamy On Sept. 4, 2013, 10:02 p.m., Virag Kothari wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13931/ > ----------------------------------------------------------- > > (Updated Sept. 4, 2013, 10:02 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1461 > https://issues.apache.org/jira/browse/OOZIE-1461 > > > Repository: oozie > > > Description > ------- > > Added sharelib service which uploads all sharelib jars containing launcher > classes to hdfs > JavaActionExecutor will add this to DC. > Sharelib service will always keep minimum 2 recent version of sharelib. Other > sharelibs older than 7 days (configurable) will be deleted. > > > Diffs > ----- > > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/DistcpActionExecutor.java > 1519337 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/HiveActionExecutor.java > 1519337 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > 1519337 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java > 1519337 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java > 1519337 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ScriptLanguageActionExecutor.java > 1519337 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ShellActionExecutor.java > 1519337 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/SqoopActionExecutor.java > 1519337 > trunk/core/src/main/java/org/apache/oozie/service/ActionService.java > 1519337 > trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java > PRE-CREATION > trunk/core/src/main/resources/oozie-default.xml 1519337 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/ActionExecutorTestCase.java > 1519337 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java > 1519337 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestShellActionExecutor.java > 1519337 > trunk/core/src/test/java/org/apache/oozie/service/TestShareLibService.java > PRE-CREATION > trunk/core/src/test/java/org/apache/oozie/test/XFsTestCase.java 1519337 > trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java 1519337 > > trunk/sharelib/hive/src/test/java/org/apache/oozie/action/hadoop/TestHiveActionExecutor.java > 1519337 > > trunk/sharelib/pig/src/test/java/org/apache/oozie/action/hadoop/TestPigActionExecutor.java > 1519337 > > trunk/sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopActionExecutor.java > 1519337 > > trunk/sharelib/streaming/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java > 1519337 > > trunk/sharelib/streaming/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutorUberJar.java > 1519337 > > Diff: https://reviews.apache.org/r/13931/diff/ > > > Testing > ------- > > Unit test case and ran pig, mapreduce examples > > > Thanks, > > Virag Kothari > >
