svn merge -c 1311972 from trunk. FIXES: MAPREDUCE-4017. Add jobname to jobsummary log (tgraves and Koji Noguchi via bobby)
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1311973 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
b46380e463
commit
b04f23fc96
|
@ -199,6 +199,9 @@ Release 0.23.3 - UNRELEASED
|
|||
MAPREDUCE-4099. ApplicationMaster may fail to remove staging directory
|
||||
(Jason Lowe via bobby)
|
||||
|
||||
MAPREDUCE-4017. Add jobname to jobsummary log (tgraves and Koji Noguchi
|
||||
via bobby)
|
||||
|
||||
Release 0.23.2 - UNRELEASED
|
||||
|
||||
INCOMPATIBLE CHANGES
|
||||
|
|
|
@ -551,6 +551,7 @@ public class JobHistoryEventHandler extends AbstractService
|
|||
summary.setUser(jse.getUserName());
|
||||
summary.setQueue(jse.getJobQueueName());
|
||||
summary.setJobSubmitTime(jse.getSubmitTime());
|
||||
summary.setJobName(jse.getJobName());
|
||||
break;
|
||||
case NORMALIZED_RESOURCE:
|
||||
NormalizedResourceEvent normalizedResourceEvent =
|
||||
|
|
|
@ -45,6 +45,7 @@ public class JobSummary {
|
|||
private long mapSlotSeconds; // TODO Not generated yet in MRV2
|
||||
private long reduceSlotSeconds; // TODO Not generated yet MRV2
|
||||
// private int clusterSlotCapacity;
|
||||
private String jobName;
|
||||
|
||||
JobSummary() {
|
||||
}
|
||||
|
@ -185,6 +186,14 @@ public class JobSummary {
|
|||
this.reduceSlotSeconds = reduceSlotSeconds;
|
||||
}
|
||||
|
||||
public String getJobName() {
|
||||
return jobName;
|
||||
}
|
||||
|
||||
public void setJobName(String jobName) {
|
||||
this.jobName = jobName;
|
||||
}
|
||||
|
||||
public String getJobSummaryString() {
|
||||
SummaryBuilder summary = new SummaryBuilder()
|
||||
.add("jobId", jobId)
|
||||
|
@ -201,7 +210,8 @@ public class JobSummary {
|
|||
.add("queue", queue)
|
||||
.add("status", jobStatus)
|
||||
.add("mapSlotSeconds", mapSlotSeconds)
|
||||
.add("reduceSlotSeconds", reduceSlotSeconds);
|
||||
.add("reduceSlotSeconds", reduceSlotSeconds)
|
||||
.add("jobName", jobName);
|
||||
return summary.toString();
|
||||
}
|
||||
|
||||
|
|
|
@ -267,6 +267,8 @@ public class TestJobHistoryParsing {
|
|||
|
||||
Assert.assertEquals("JobId does not match", jobId.toString(),
|
||||
jobSummaryElements.get("jobId"));
|
||||
Assert.assertEquals("JobName does not match", "test",
|
||||
jobSummaryElements.get("jobName"));
|
||||
Assert.assertTrue("submitTime should not be 0",
|
||||
Long.parseLong(jobSummaryElements.get("submitTime")) != 0);
|
||||
Assert.assertTrue("launchTime should not be 0",
|
||||
|
|
Loading…
Reference in New Issue