Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

YARN-7611. Node manager web UI should display container type in containers page. Contributed by Weiwei Yang. 

Previous commit 218d622529 introduces a compile error,
  so it was reverted by commit db74ada80f.
  This commit fixed the issue.

(cherry picked from commit 05c347fe51)
This commit is contained in:
Weiwei Yang 2017-12-06 12:21:52 +08:00
parent db74ada80f
commit 42deb54049
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/webapp/AllContainersPage.java
View File

@ -51,9 +51,9 @@ public class AllContainersPage extends NMView {
private String containersTableInit() {
return tableInit().
// containerid, containerid, log-url
// containerid, executiontype, containerid, log-url
append(", aoColumns:[").append(getContainersIdColumnDefs())
.append(", null, {bSearchable:false}]} ").toString();
.append(", null, null, {bSearchable:false}]} ").toString();
}
private String getContainersIdColumnDefs() {
@ -83,6 +83,7 @@ public class AllContainersPage extends NMView {
.thead()
.tr()
.td()._("ContainerId")._()
.td()._("ExecutionType")._()
.td()._("ContainerState")._()
.td()._("logs")._()
._()
@ -94,6 +95,7 @@ public class AllContainersPage extends NMView {
.tr()
.td().a(url("container", info.getId()), info.getId())
._()
.td()._(info.getExecutionType())._()
.td()._(info.getState())._()
.td()
.a(url(info.getShortLogLink()), "logs")._()