Browse Source

YARN 2163. WebUI: Order of AppId in apps table should be consistent with ApplicationId.compareTo()

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1605964 13f79535-47bb-0310-9956-ffa450edef68
Ravi Prakash 11 years ago
parent
commit
f084170a0a

+ 3 - 0
hadoop-yarn-project/CHANGES.txt

@@ -293,6 +293,9 @@ Release 2.5.0 - UNRELEASED
     YARN-2204. TestAMRestart#testAMRestartWithExistingContainers assumes
     YARN-2204. TestAMRestart#testAMRestartWithExistingContainers assumes
     CapacityScheduler. (Robert Kanter via kasha)
     CapacityScheduler. (Robert Kanter via kasha)
 
 
+    YARN-2163. WebUI: Order of AppId in apps table should be consistent with
+    ApplicationId.compareTo(). (Wangda Tan via raviprak)
+
 Release 2.4.1 - 2014-06-23 
 Release 2.4.1 - 2014-06-23 
 
 
   INCOMPATIBLE CHANGES
   INCOMPATIBLE CHANGES

+ 1 - 7
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/webapps/static/yarn.dt.plugins.js

@@ -119,13 +119,7 @@ function parseHadoopID(data, type, full) {
     return data;
     return data;
   }
   }
   //Return the visible string rather than the entire HTML tag
   //Return the visible string rather than the entire HTML tag
-  if (type === 'filter') {
-    return data.split('>')[1].split('<')[0];
-  }
-  //Parse the ID for 'sort', 'type' and undefined
-  //The number after the last '_' and before the end tag '<'
-  var splits = data.split('_');
-  return splits[parseInt(splits.length-1)].split('<')[0];
+  return data.split('>')[1].split('<')[0];
 }
 }
 
 
 //JSON array element is "20000 attempt_1360183373897_0001_m_000002_0"
 //JSON array element is "20000 attempt_1360183373897_0001_m_000002_0"

+ 1 - 1
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RmView.java

@@ -77,7 +77,7 @@ public class RmView extends TwoColumnLayout {
     StringBuilder sb = new StringBuilder();
     StringBuilder sb = new StringBuilder();
     return sb
     return sb
       .append("[\n")
       .append("[\n")
-      .append("{'sType':'numeric', 'aTargets': [0]")
+      .append("{'sType':'string', 'aTargets': [0]")
       .append(", 'mRender': parseHadoopID }")
       .append(", 'mRender': parseHadoopID }")
 
 
       .append("\n, {'sType':'numeric', 'aTargets': [5, 6]")
       .append("\n, {'sType':'numeric', 'aTargets': [5, 6]")