You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by om...@apache.org on 2007/09/19 08:59:17 UTC

svn commit: r577171 - in /lucene/hadoop/trunk: CHANGES.txt src/java/org/apache/hadoop/mapred/JobClient.java

Author: omalley
Date: Tue Sep 18 23:59:16 2007
New Revision: 577171

URL: http://svn.apache.org/viewvc?rev=577171&view=rev
Log:
HADOOP-1907.  Fixes null pointer exception while getting the task diagnostics 
in the JobClient. Contributed by Christian Kunz.

Modified:
    lucene/hadoop/trunk/CHANGES.txt
    lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobClient.java

Modified: lucene/hadoop/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/CHANGES.txt?rev=577171&r1=577170&r2=577171&view=diff
==============================================================================
--- lucene/hadoop/trunk/CHANGES.txt (original)
+++ lucene/hadoop/trunk/CHANGES.txt Tue Sep 18 23:59:16 2007
@@ -152,6 +152,9 @@
     HADOOP-1897.  Fix links to about.html page from other pages in 
     the web site. (enis)
 
+    HADOOP-1907.  Fix null pointer exception when getting task diagnostics
+    in JobClient. (Christian Kunz via omalley)
+
   IMPROVEMENTS
 
     HADOOP-1266. Remove dependency of package org.apache.hadoop.net on 

Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobClient.java
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobClient.java?rev=577171&r1=577170&r2=577171&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobClient.java (original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobClient.java Tue Sep 18 23:59:16 2007
@@ -648,8 +648,10 @@
                   String[] taskDiagnostics = 
                 	  jc.jobSubmitClient.getTaskDiagnostics(jobId, tipId, 
                 			                                taskId); 
-                  for(String diagnostics : taskDiagnostics){
-                   	System.err.println(diagnostics);
+                  if (taskDiagnostics != null) {
+                    for(String diagnostics : taskDiagnostics){
+                   	  System.err.println(diagnostics);
+                    }
                   }
                   // Displaying the task logs
                   displayTaskLogs(event.getTaskId(), event.getTaskTrackerHttp());