瀏覽代碼

svn merge -c 1367601 FIXES: MAPREDUCE-4496. AM logs link is missing user name (Jason Lowe via bobby)

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1367602 13f79535-47bb-0310-9956-ffa450edef68
Robert Joseph Evans 12 年之前
父節點
當前提交
6fe821a14a

+ 2 - 0
hadoop-mapreduce-project/CHANGES.txt

@@ -645,6 +645,8 @@ Release 0.23.3 - UNRELEASED
     MAPREDUCE-4456. LocalDistributedCacheManager can get an 
     MAPREDUCE-4456. LocalDistributedCacheManager can get an 
     ArrayIndexOutOfBounds when creating symlinks (Robert Evans via tgraves)
     ArrayIndexOutOfBounds when creating symlinks (Robert Evans via tgraves)
 
 
+    MAPREDUCE-4496. AM logs link is missing user name (Jason Lowe via bobby)
+
 Release 0.23.2 - UNRELEASED
 Release 0.23.2 - UNRELEASED
 
 
   INCOMPATIBLE CHANGES
   INCOMPATIBLE CHANGES

+ 1 - 1
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/dao/AMAttemptInfo.java

@@ -64,7 +64,7 @@ public class AMAttemptInfo {
     if (containerId != null) {
     if (containerId != null) {
       this.containerId = containerId.toString();
       this.containerId = containerId.toString();
       this.logsLink = join("http://" + nodeHttpAddress,
       this.logsLink = join("http://" + nodeHttpAddress,
-          ujoin("node", "containerlogs", this.containerId));
+          ujoin("node", "containerlogs", this.containerId, user));
     }
     }
   }
   }
 
 

+ 2 - 1
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/v2/app/webapp/TestAMWebServicesJobs.java

@@ -972,7 +972,8 @@ public class TestAMWebServicesJobs extends JerseyTest {
         WebServicesTestUtils.checkStringMatch("containerId", amInfo
         WebServicesTestUtils.checkStringMatch("containerId", amInfo
             .getContainerId().toString(), containerId);
             .getContainerId().toString(), containerId);
 
 
-        String localLogsLink = ujoin("node", "containerlogs", containerId);
+        String localLogsLink =ujoin("node", "containerlogs", containerId,
+            job.getUserName());
 
 
         assertTrue("logsLink", logsLink.contains(localLogsLink));
         assertTrue("logsLink", logsLink.contains(localLogsLink));
       }
       }