HADOOP-749. Fix a NullPointerException in jobfailures.jsp.

git-svn-id: https://svn.apache.org/repos/asf/lucene/hadoop/trunk@480280 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -128,6 +128,9 @@ Trunk (unreleased changes)
 37. HADOOP-739.  Fix TestIPC to use different port number, making it
     more reliable.  (Nigel Daley via cutting)
 
+38. HADOOP-749.  Fix a NullPointerException in jobfailures.jsp.
+    (omalley via cutting)
+
 
 Release 0.8.0 - 2006-11-03
 

src/webapps/job/jobfailures.jsp

@@ -49,15 +49,20 @@
         }
         out.print("</pre></td>");
         
-        String taskLogUrl = "http://" + taskTracker.getHost() + ":" +
-        	taskTracker.getHttpPort() + "/tasklog.jsp?taskid=" + statuses[i].getTaskId();
-        String tailFourKBUrl = taskLogUrl + "&tail=true&tailsize=4096";
-        String tailEightKBUrl = taskLogUrl + "&tail=true&tailsize=8192";
-        String entireLogUrl = taskLogUrl + "&all=true";
         out.print("<td>");
-        out.print("<a href=\"" + tailFourKBUrl + "\">Last 4KB</a><br/>");
-        out.print("<a href=\"" + tailEightKBUrl + "\">Last 8KB</a><br/>");
-        out.print("<a href=\"" + entireLogUrl + "\">All</a><br/>");
+        if (taskTracker != null) {
+          String taskLogUrl = "http://" + taskTracker.getHost() + ":" +
+          	taskTracker.getHttpPort() + "/tasklog.jsp?taskid=" + 
+          	statuses[i].getTaskId();
+          String tailFourKBUrl = taskLogUrl + "&tail=true&tailsize=4096";
+          String tailEightKBUrl = taskLogUrl + "&tail=true&tailsize=8192";
+          String entireLogUrl = taskLogUrl + "&all=true";
+          out.print("<a href=\"" + tailFourKBUrl + "\">Last 4KB</a><br/>");
+          out.print("<a href=\"" + tailEightKBUrl + "\">Last 8KB</a><br/>");
+          out.print("<a href=\"" + entireLogUrl + "\">All</a><br/>");
+        } else { 
+          out.print("n/a"); // task tracker was lost
+        }
         out.print("</td>");
         
         out.print("</tr>\n");

src/webapps/job/taskdetails.jsp

@@ -88,21 +88,20 @@
         }
       }
       out.print("</pre></td>");
+      out.print("<td>");
       if (taskAttemptTracker == null) {
-        out.print("&nbsp;");
+        out.print("n/a");
       } else {
         String taskLogUrl = taskAttemptTracker + "/tasklog.jsp?taskid=" + 
                               status.getTaskId();
         String tailFourKBUrl = taskLogUrl + "&tail=true&tailsize=4096";
         String tailEightKBUrl = taskLogUrl + "&tail=true&tailsize=8192";
         String entireLogUrl = taskLogUrl + "&all=true";
-        out.print("<td>");
         out.print("<a href=\"" + tailFourKBUrl + "\">Last 4KB</a><br/>");
         out.print("<a href=\"" + tailEightKBUrl + "\">Last 8KB</a><br/>");
         out.print("<a href=\"" + entireLogUrl + "\">All</a><br/>");
-        out.print("</td>");
       }
-      out.println("</tr>\n");
+      out.println("</td></tr>\n");
     }
   }
   %>