svn commit: r1442275 - in /oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop: LauncherMapper.java PigActionExecutor.java

[rkanter]

Author: rkanter
Date: Mon Feb  4 18:03:53 2013
New Revision: 1442275

URL: http://svn.apache.org/viewvc?rev=1442275&view=rev
Log:
OOZIE-1160 additional minor review comments

Modified:
    
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
    
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java

Modified: 
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java?rev=1442275&r1=1442274&r2=1442275&view=diff
==============================================================================
--- 
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
 (original)
+++ 
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
 Mon Feb  4 18:03:53 2013
@@ -547,7 +547,6 @@ public class LauncherMapper<K1, V1, K2, 
                             System.out.println();
                         }
                         handleActionStatsData(reporter);
-                        handleExternalChildIDs(reporter);
                         File newId = new 
File(System.getProperty("oozie.action.newId.properties"));
                         if (newId.exists()) {
                             Properties props = new Properties();

Modified: 
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java?rev=1442275&r1=1442274&r2=1442275&view=diff
==============================================================================
--- 
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java
 (original)
+++ 
oozie/branches/branch-3.3/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java
 Mon Feb  4 18:03:53 2013
@@ -174,10 +174,7 @@ public class PigActionExecutor extends J
     protected void setActionCompletionData(Context context, FileSystem fs) 
throws HadoopAccessorException, IOException,
             URISyntaxException {
         String data = getExternalChildIDs(context, fs);
-        if (data != null && !data.isEmpty()) {
-            context.setExternalChildIDs(data);
-            XLog.getLog(getClass()).info(XLog.STD, "Hadoop Jobs launched : 
[{0}]", data);
-        }
+        context.setExternalChildIDs(data);
     }
 
     private String getExternalChildIDs(Context context, FileSystem actionFs) 
throws IOException,
@@ -186,6 +183,7 @@ public class PigActionExecutor extends J
         String externalIDs = null;
         if (actionFs.exists(actionOutput)) {
             externalIDs = getDataFromPath(actionOutput, actionFs);
+            XLog.getLog(getClass()).info(XLog.STD, "Hadoop Jobs launched : 
[{0}]", externalIDs);
         }
         return externalIDs;
     }