MAPREDUCE-3541. Fix broken TestJobQueueClient test. (Ravi Prakash via mahadev) - Merging r1214421 from trunk

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-0.23@1214434 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Mahadev Konar 2011-12-14 20:00:27 +00:00
parent 4ba1442b67
commit 1430c33b6d
4 changed files with 20 additions and 11 deletions

View File

@ -242,6 +242,9 @@ Release 0.23.1 - Unreleased
MAPREDUCE-3557. MR1 test fail to compile because of missing hadoop-archives
dependency. (tucu)
MAPREDUCE-3541. Fix broken TestJobQueueClient test. (Ravi Prakash via
mahadev)
MAPREDUCE-3398. Fixed log aggregation to work correctly in secure mode.
(Siddharth Seth via vinodkv)

View File

@ -110,6 +110,13 @@ class JobQueueClient extends Configured implements Tool {
}
// format and print information about the passed in job queue.
void printJobQueueInfo(JobQueueInfo jobQueueInfo, Writer writer)
throws IOException {
printJobQueueInfo(jobQueueInfo, writer, "");
}
// format and print information about the passed in job queue.
@SuppressWarnings("deprecation")
void printJobQueueInfo(JobQueueInfo jobQueueInfo, Writer writer,
String prefix) throws IOException {
if (jobQueueInfo == null) {
@ -136,7 +143,7 @@ class JobQueueClient extends Configured implements Tool {
private void displayQueueList() throws IOException {
JobQueueInfo[] rootQueues = jc.getRootQueues();
for (JobQueueInfo queue : rootQueues) {
printJobQueueInfo(queue, new PrintWriter(System.out), "");
printJobQueueInfo(queue, new PrintWriter(System.out));
}
}
@ -174,7 +181,7 @@ class JobQueueClient extends Configured implements Tool {
System.out.println("Queue \"" + queue + "\" does not exist.");
return;
}
printJobQueueInfo(jobQueueInfo, new PrintWriter(System.out), "");
printJobQueueInfo(jobQueueInfo, new PrintWriter(System.out));
if (showJobs && (jobQueueInfo.getChildren() == null ||
jobQueueInfo.getChildren().size() == 0)) {
JobStatus[] jobs = jc.getJobsFromQueue(queue);

View File

@ -45,7 +45,7 @@ public class TestJobQueueClient {
ByteArrayOutputStream bbos = new ByteArrayOutputStream();
PrintWriter writer = new PrintWriter(bbos);
queueClient.printJobQueueInfo(parent, writer, "");
queueClient.printJobQueueInfo(parent, writer);
Assert.assertTrue("printJobQueueInfo did not print grandchild's name",
bbos.toString().contains("GrandChildQueue"));

View File

@ -30,6 +30,8 @@ import java.io.StringWriter;
import java.util.ArrayList;
import java.util.List;
import junit.framework.Assert;
import org.apache.hadoop.mapreduce.QueueInfo;
import org.junit.After;
import org.junit.Test;
@ -79,14 +81,11 @@ public class TestJobQueueClient {
StringWriter writer = new StringWriter();
client.printJobQueueInfo(root, writer);
StringBuffer sb = new StringBuffer();
sb.append("Queue Name : q1 \n");
sb.append("Queue State : running \n");
sb.append("Scheduling Info : q1 scheduling info \n");
sb.append("Child Queues : q1:1, q1:2\n");
sb.append("======================\n");
assertEquals(sb.toString(), writer.toString());
Assert.assertTrue(writer.toString().contains("Queue Name : q1"));
Assert.assertTrue(writer.toString().contains("Queue State : running"));
Assert.assertTrue(writer.toString().contains("Scheduling Info : q1 scheduling info"));
Assert.assertTrue(writer.toString().contains("Queue Name : q1:1"));
Assert.assertTrue(writer.toString().contains("Queue Name : q1:2"));
}
@Test