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

YARN-5094. some YARN container events have timestamp of -1. 

(cherry picked from commit 4aca4ff759)
This commit is contained in:
Haibo Chen 2018-01-19 10:27:02 -08:00
parent 6c10e0e12b
commit 8793e45f53
4 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/application/ApplicationEvent.java
View File

@ -27,7 +27,7 @@ public class ApplicationEvent extends AbstractEvent<ApplicationEventType> {
public ApplicationEvent(ApplicationId appID,
ApplicationEventType appEventType) {
super(appEventType);
super(appEventType, System.currentTimeMillis());
this.applicationID = appID;
}

2
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/container/ContainerEvent.java
View File

@ -27,7 +27,7 @@ public class ContainerEvent extends AbstractEvent<ContainerEventType> {
private final ContainerId containerID;
public ContainerEvent(ContainerId cID, ContainerEventType eventType) {
super(eventType);
super(eventType, System.currentTimeMillis());
this.containerID = cID;
}

2
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/localizer/event/LocalizationEvent.java
View File

@ -27,7 +27,7 @@ import org.apache.hadoop.yarn.server.nodemanager.containermanager.localizer.Reso
public class LocalizationEvent extends AbstractEvent<LocalizationEventType> {
public LocalizationEvent(LocalizationEventType event) {
super(event);
super(event, System.currentTimeMillis());
}
}

6
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/application/TestApplication.java
View File

@ -262,7 +262,8 @@ public class TestApplication {
verify(wa.localizerBus).handle(
refEq(new ApplicationLocalizationEvent(
LocalizationEventType.DESTROY_APPLICATION_RESOURCES, wa.app)));
LocalizationEventType.DESTROY_APPLICATION_RESOURCES,
wa.app), "timestamp"));
verify(wa.auxBus).handle(
refEq(new AuxServicesEvent(
@ -341,7 +342,8 @@ public class TestApplication {
verify(wa.localizerBus).handle(
refEq(new ApplicationLocalizationEvent(
LocalizationEventType.DESTROY_APPLICATION_RESOURCES, wa.app)));
LocalizationEventType.DESTROY_APPLICATION_RESOURCES, wa.app),
"timestamp"));
wa.appResourcesCleanedup();
for ( Container container : wa.containers) {