You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by cn...@apache.org on 2014/11/04 05:04:12 UTC
git commit: HDFS-7328. TestTraceAdmin assumes Unix line endings.
Contributed by Chris Nauroth.
Repository: hadoop
Updated Branches:
refs/heads/trunk c5a46d4c8 -> 2bb327eb9
HDFS-7328. TestTraceAdmin assumes Unix line endings. Contributed by Chris Nauroth.
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/2bb327eb
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/2bb327eb
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/2bb327eb
Branch: refs/heads/trunk
Commit: 2bb327eb939f57626d3dac10f7016ed634375d94
Parents: c5a46d4
Author: cnauroth <cn...@apache.org>
Authored: Mon Nov 3 19:59:52 2014 -0800
Committer: cnauroth <cn...@apache.org>
Committed: Mon Nov 3 19:59:52 2014 -0800
----------------------------------------------------------------------
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 2 ++
.../java/org/apache/hadoop/tracing/TestTraceAdmin.java | 13 +++++++------
2 files changed, 9 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/2bb327eb/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
index dfe8f4d..be7b9bf 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -1356,6 +1356,8 @@ Release 2.6.0 - UNRELEASED
HDFS-7274. Disable SSLv3 in HttpFS. (Robert Kanter via kasha)
+ HDFS-7328. TestTraceAdmin assumes Unix line endings. (cnauroth)
+
Release 2.5.1 - 2014-09-05
INCOMPATIBLE CHANGES
http://git-wip-us.apache.org/repos/asf/hadoop/blob/2bb327eb/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/tracing/TestTraceAdmin.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/tracing/TestTraceAdmin.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/tracing/TestTraceAdmin.java
index 79a1f39..77860ba 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/tracing/TestTraceAdmin.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/tracing/TestTraceAdmin.java
@@ -28,6 +28,7 @@ import java.io.File;
import java.io.PrintStream;
public class TestTraceAdmin {
+ private static final String NEWLINE = System.getProperty("line.separator");
private String runTraceCommand(TraceAdmin trace, String... cmd)
throws Exception {
@@ -66,10 +67,10 @@ public class TestTraceAdmin {
try {
TraceAdmin trace = new TraceAdmin();
trace.setConf(conf);
- Assert.assertEquals("ret:0, [no span receivers found]\n",
+ Assert.assertEquals("ret:0, [no span receivers found]" + NEWLINE,
runTraceCommand(trace, "-list", "-host", getHostPortForNN(cluster)));
Assert.assertEquals("ret:0, Added trace span receiver 1 with " +
- "configuration local-file-span-receiver.path = " + tracePath + "\n",
+ "configuration local-file-span-receiver.path = " + tracePath + NEWLINE,
runTraceCommand(trace, "-add", "-host", getHostPortForNN(cluster),
"-class", "org.htrace.impl.LocalFileSpanReceiver",
"-Clocal-file-span-receiver.path=" + tracePath));
@@ -77,17 +78,17 @@ public class TestTraceAdmin {
runTraceCommand(trace, "-list", "-host", getHostPortForNN(cluster));
Assert.assertTrue(list.startsWith("ret:0"));
Assert.assertTrue(list.contains("1 org.htrace.impl.LocalFileSpanReceiver"));
- Assert.assertEquals("ret:0, Removed trace span receiver 1\n",
+ Assert.assertEquals("ret:0, Removed trace span receiver 1" + NEWLINE,
runTraceCommand(trace, "-remove", "1", "-host",
getHostPortForNN(cluster)));
- Assert.assertEquals("ret:0, [no span receivers found]\n",
+ Assert.assertEquals("ret:0, [no span receivers found]" + NEWLINE,
runTraceCommand(trace, "-list", "-host", getHostPortForNN(cluster)));
Assert.assertEquals("ret:0, Added trace span receiver 2 with " +
- "configuration local-file-span-receiver.path = " + tracePath + "\n",
+ "configuration local-file-span-receiver.path = " + tracePath + NEWLINE,
runTraceCommand(trace, "-add", "-host", getHostPortForNN(cluster),
"-class", "LocalFileSpanReceiver",
"-Clocal-file-span-receiver.path=" + tracePath));
- Assert.assertEquals("ret:0, Removed trace span receiver 2\n",
+ Assert.assertEquals("ret:0, Removed trace span receiver 2" + NEWLINE,
runTraceCommand(trace, "-remove", "2", "-host",
getHostPortForNN(cluster)));
} finally {