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 2015/04/03 19:54:19 UTC
[1/3] hadoop git commit: HDFS-8039. Fix
TestDebugAdmin#testRecoverLease and testVerfiyBlockChecksumCommand on
Windows. Contributed by Xiaoyu Yao.
Repository: hadoop
Updated Branches:
refs/heads/branch-2 3757e731e -> 28e0602e7
refs/heads/branch-2.7 b08bc43c9 -> 6855bbc6c
refs/heads/trunk a0795c1e8 -> db80e4289
HDFS-8039. Fix TestDebugAdmin#testRecoverLease and testVerfiyBlockChecksumCommand on Windows. Contributed by Xiaoyu Yao.
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/db80e428
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/db80e428
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/db80e428
Branch: refs/heads/trunk
Commit: db80e428919e4e763f56b9344c748e7f8fb46491
Parents: a0795c1
Author: cnauroth <cn...@apache.org>
Authored: Fri Apr 3 10:44:02 2015 -0700
Committer: cnauroth <cn...@apache.org>
Committed: Fri Apr 3 10:44:02 2015 -0700
----------------------------------------------------------------------
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 3 +++
.../org/apache/hadoop/hdfs/tools/TestDebugAdmin.java | 15 ++++++++-------
2 files changed, 11 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/db80e428/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 366d8fb..ae56f1b 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -1370,6 +1370,9 @@ Release 2.7.0 - UNRELEASED
TestDFSClientRetries#testDFSClientConfigurationLocateFollowingBlockInitialDelay
for Windows. (Xiaoyu Yao via cnauroth)
+ HDFS-8039. Fix TestDebugAdmin#testRecoverLease and
+ testVerifyBlockChecksumCommand on Windows. (Xiaoyu Yao via cnauroth)
+
BREAKDOWN OF HDFS-7584 SUBTASKS AND RELATED JIRAS
HDFS-7720. Quota by Storage Type API, tools and ClientNameNode
http://git-wip-us.apache.org/repos/asf/hadoop/blob/db80e428/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
index 44b6ba9..52b194d 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
@@ -77,17 +77,18 @@ public class TestDebugAdmin {
System.setOut(oldOut);
IOUtils.closeStream(out);
}
- return "ret: " + ret + ", " + bytes.toString();
+ return "ret: " + ret + ", " +
+ bytes.toString().replaceAll(System.getProperty("line.separator"), "");
}
@Test(timeout = 60000)
public void testRecoverLease() throws Exception {
- assertEquals("ret: 1, You must supply a -path argument to recoverLease.\n",
+ assertEquals("ret: 1, You must supply a -path argument to recoverLease.",
runCmd(new String[]{"recoverLease", "-retries", "1"}));
FSDataOutputStream out = fs.create(new Path("/foo"));
out.write(123);
out.close();
- assertEquals("ret: 0, recoverLease SUCCEEDED on /foo\n",
+ assertEquals("ret: 0, recoverLease SUCCEEDED on /foo",
runCmd(new String[]{"recoverLease", "-path", "/foo"}));
}
@@ -98,18 +99,18 @@ public class TestDebugAdmin {
ExtendedBlock block = DFSTestUtil.getFirstBlock(fs, new Path("/bar"));
File blockFile = getBlockFile(fsd,
block.getBlockPoolId(), block.getLocalBlock());
- assertEquals("ret: 1, You must specify a meta file with -meta\n",
+ assertEquals("ret: 1, You must specify a meta file with -meta",
runCmd(new String[]{"verify", "-block", blockFile.getAbsolutePath()}));
File metaFile = getMetaFile(fsd,
block.getBlockPoolId(), block.getLocalBlock());
assertEquals("ret: 0, Checksum type: " +
- "DataChecksum(type=CRC32C, chunkSize=512)\n",
+ "DataChecksum(type=CRC32C, chunkSize=512)",
runCmd(new String[]{"verify",
"-meta", metaFile.getAbsolutePath()}));
assertEquals("ret: 0, Checksum type: " +
- "DataChecksum(type=CRC32C, chunkSize=512)\n" +
+ "DataChecksum(type=CRC32C, chunkSize=512)" +
"Checksum verification succeeded on block file " +
- blockFile.getAbsolutePath() + "\n",
+ blockFile.getAbsolutePath(),
runCmd(new String[]{"verify",
"-meta", metaFile.getAbsolutePath(),
"-block", blockFile.getAbsolutePath()})
[3/3] hadoop git commit: HDFS-8039. Fix
TestDebugAdmin#testRecoverLease and testVerfiyBlockChecksumCommand on
Windows. Contributed by Xiaoyu Yao.
Posted by cn...@apache.org.
HDFS-8039. Fix TestDebugAdmin#testRecoverLease and testVerfiyBlockChecksumCommand on Windows. Contributed by Xiaoyu Yao.
(cherry picked from commit db80e428919e4e763f56b9344c748e7f8fb46491)
(cherry picked from commit 28e0602e7bf20581e3ce477512c60a854a4612d3)
Conflicts:
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/6855bbc6
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/6855bbc6
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/6855bbc6
Branch: refs/heads/branch-2.7
Commit: 6855bbc6ca17a730e8284214b676b5ca28a36727
Parents: b08bc43
Author: cnauroth <cn...@apache.org>
Authored: Fri Apr 3 10:44:02 2015 -0700
Committer: cnauroth <cn...@apache.org>
Committed: Fri Apr 3 10:45:15 2015 -0700
----------------------------------------------------------------------
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 3 +++
.../org/apache/hadoop/hdfs/tools/TestDebugAdmin.java | 15 ++++++++-------
2 files changed, 11 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/6855bbc6/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 1a7e8bb..e9343c2 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -956,6 +956,9 @@ Release 2.7.0 - UNRELEASED
HDFS-7954. TestBalancer#testBalancerWithPinnedBlocks should not be executed
on Windows. (Xiaoyu Yao via szetszwo)
+ HDFS-8039. Fix TestDebugAdmin#testRecoverLease and
+ testVerifyBlockChecksumCommand on Windows. (Xiaoyu Yao via cnauroth)
+
BREAKDOWN OF HDFS-7584 SUBTASKS AND RELATED JIRAS
HDFS-7720. Quota by Storage Type API, tools and ClientNameNode
http://git-wip-us.apache.org/repos/asf/hadoop/blob/6855bbc6/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
index 44b6ba9..52b194d 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
@@ -77,17 +77,18 @@ public class TestDebugAdmin {
System.setOut(oldOut);
IOUtils.closeStream(out);
}
- return "ret: " + ret + ", " + bytes.toString();
+ return "ret: " + ret + ", " +
+ bytes.toString().replaceAll(System.getProperty("line.separator"), "");
}
@Test(timeout = 60000)
public void testRecoverLease() throws Exception {
- assertEquals("ret: 1, You must supply a -path argument to recoverLease.\n",
+ assertEquals("ret: 1, You must supply a -path argument to recoverLease.",
runCmd(new String[]{"recoverLease", "-retries", "1"}));
FSDataOutputStream out = fs.create(new Path("/foo"));
out.write(123);
out.close();
- assertEquals("ret: 0, recoverLease SUCCEEDED on /foo\n",
+ assertEquals("ret: 0, recoverLease SUCCEEDED on /foo",
runCmd(new String[]{"recoverLease", "-path", "/foo"}));
}
@@ -98,18 +99,18 @@ public class TestDebugAdmin {
ExtendedBlock block = DFSTestUtil.getFirstBlock(fs, new Path("/bar"));
File blockFile = getBlockFile(fsd,
block.getBlockPoolId(), block.getLocalBlock());
- assertEquals("ret: 1, You must specify a meta file with -meta\n",
+ assertEquals("ret: 1, You must specify a meta file with -meta",
runCmd(new String[]{"verify", "-block", blockFile.getAbsolutePath()}));
File metaFile = getMetaFile(fsd,
block.getBlockPoolId(), block.getLocalBlock());
assertEquals("ret: 0, Checksum type: " +
- "DataChecksum(type=CRC32C, chunkSize=512)\n",
+ "DataChecksum(type=CRC32C, chunkSize=512)",
runCmd(new String[]{"verify",
"-meta", metaFile.getAbsolutePath()}));
assertEquals("ret: 0, Checksum type: " +
- "DataChecksum(type=CRC32C, chunkSize=512)\n" +
+ "DataChecksum(type=CRC32C, chunkSize=512)" +
"Checksum verification succeeded on block file " +
- blockFile.getAbsolutePath() + "\n",
+ blockFile.getAbsolutePath(),
runCmd(new String[]{"verify",
"-meta", metaFile.getAbsolutePath(),
"-block", blockFile.getAbsolutePath()})
[2/3] hadoop git commit: HDFS-8039. Fix
TestDebugAdmin#testRecoverLease and testVerfiyBlockChecksumCommand on
Windows. Contributed by Xiaoyu Yao.
Posted by cn...@apache.org.
HDFS-8039. Fix TestDebugAdmin#testRecoverLease and testVerfiyBlockChecksumCommand on Windows. Contributed by Xiaoyu Yao.
(cherry picked from commit db80e428919e4e763f56b9344c748e7f8fb46491)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/28e0602e
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/28e0602e
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/28e0602e
Branch: refs/heads/branch-2
Commit: 28e0602e7bf20581e3ce477512c60a854a4612d3
Parents: 3757e73
Author: cnauroth <cn...@apache.org>
Authored: Fri Apr 3 10:44:02 2015 -0700
Committer: cnauroth <cn...@apache.org>
Committed: Fri Apr 3 10:44:32 2015 -0700
----------------------------------------------------------------------
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 3 +++
.../org/apache/hadoop/hdfs/tools/TestDebugAdmin.java | 15 ++++++++-------
2 files changed, 11 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/28e0602e/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 10a1361..c6d32d0 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
+++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
@@ -1069,6 +1069,9 @@ Release 2.7.0 - UNRELEASED
TestDFSClientRetries#testDFSClientConfigurationLocateFollowingBlockInitialDelay
for Windows. (Xiaoyu Yao via cnauroth)
+ HDFS-8039. Fix TestDebugAdmin#testRecoverLease and
+ testVerifyBlockChecksumCommand on Windows. (Xiaoyu Yao via cnauroth)
+
BREAKDOWN OF HDFS-7584 SUBTASKS AND RELATED JIRAS
HDFS-7720. Quota by Storage Type API, tools and ClientNameNode
http://git-wip-us.apache.org/repos/asf/hadoop/blob/28e0602e/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
index 44b6ba9..52b194d 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDebugAdmin.java
@@ -77,17 +77,18 @@ public class TestDebugAdmin {
System.setOut(oldOut);
IOUtils.closeStream(out);
}
- return "ret: " + ret + ", " + bytes.toString();
+ return "ret: " + ret + ", " +
+ bytes.toString().replaceAll(System.getProperty("line.separator"), "");
}
@Test(timeout = 60000)
public void testRecoverLease() throws Exception {
- assertEquals("ret: 1, You must supply a -path argument to recoverLease.\n",
+ assertEquals("ret: 1, You must supply a -path argument to recoverLease.",
runCmd(new String[]{"recoverLease", "-retries", "1"}));
FSDataOutputStream out = fs.create(new Path("/foo"));
out.write(123);
out.close();
- assertEquals("ret: 0, recoverLease SUCCEEDED on /foo\n",
+ assertEquals("ret: 0, recoverLease SUCCEEDED on /foo",
runCmd(new String[]{"recoverLease", "-path", "/foo"}));
}
@@ -98,18 +99,18 @@ public class TestDebugAdmin {
ExtendedBlock block = DFSTestUtil.getFirstBlock(fs, new Path("/bar"));
File blockFile = getBlockFile(fsd,
block.getBlockPoolId(), block.getLocalBlock());
- assertEquals("ret: 1, You must specify a meta file with -meta\n",
+ assertEquals("ret: 1, You must specify a meta file with -meta",
runCmd(new String[]{"verify", "-block", blockFile.getAbsolutePath()}));
File metaFile = getMetaFile(fsd,
block.getBlockPoolId(), block.getLocalBlock());
assertEquals("ret: 0, Checksum type: " +
- "DataChecksum(type=CRC32C, chunkSize=512)\n",
+ "DataChecksum(type=CRC32C, chunkSize=512)",
runCmd(new String[]{"verify",
"-meta", metaFile.getAbsolutePath()}));
assertEquals("ret: 0, Checksum type: " +
- "DataChecksum(type=CRC32C, chunkSize=512)\n" +
+ "DataChecksum(type=CRC32C, chunkSize=512)" +
"Checksum verification succeeded on block file " +
- blockFile.getAbsolutePath() + "\n",
+ blockFile.getAbsolutePath(),
runCmd(new String[]{"verify",
"-meta", metaFile.getAbsolutePath(),
"-block", blockFile.getAbsolutePath()})