Repository: hadoop Updated Branches: refs/heads/branch-2.8 71a3b01d5 -> dde3eaed6
HDFS-10783. The option '-maxSize' and '-step' fail in OfflineImageViewer. Contributed by Yiqun Lin. (cherry picked from commit e90f3359de299ef5e3a54ca71070e3dfe1dbb98c) (cherry picked from commit 94a844b4f61d4a47b2a5885e3db065a07ed29d97) Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/dde3eaed Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/dde3eaed Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/dde3eaed Branch: refs/heads/branch-2.8 Commit: dde3eaed629ea1849a67c14648c50a131f9d4f09 Parents: 71a3b01 Author: Akira Ajisaka <aajis...@apache.org> Authored: Tue Aug 23 19:56:27 2016 +0900 Committer: Akira Ajisaka <aajis...@apache.org> Committed: Tue Aug 23 19:59:07 2016 +0900 ---------------------------------------------------------------------- .../offlineImageViewer/OfflineImageViewer.java | 2 ++ .../TestOfflineImageViewer.java | 19 +++++++++++++++++++ 2 files changed, 21 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/dde3eaed/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/OfflineImageViewer.java ---------------------------------------------------------------------- diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/OfflineImageViewer.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/OfflineImageViewer.java index f1dd959..630e4f0 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/OfflineImageViewer.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/OfflineImageViewer.java @@ -186,6 +186,8 @@ public class OfflineImageViewer { options.addOption("p", "processor", true, ""); options.addOption("h", "help", false, ""); + options.addOption("maxSize", true, ""); + options.addOption("step", true, ""); options.addOption("skipBlocks", false, ""); options.addOption("printToScreen", false, ""); options.addOption("delimiter", true, ""); http://git-wip-us.apache.org/repos/asf/hadoop/blob/dde3eaed/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/TestOfflineImageViewer.java ---------------------------------------------------------------------- diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/TestOfflineImageViewer.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/TestOfflineImageViewer.java index b9aa7f3..a7c30ec 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/TestOfflineImageViewer.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/TestOfflineImageViewer.java @@ -601,4 +601,23 @@ public class TestOfflineImageViewer { "FileDistribution", "-maxSize", "23", "-step", "4"}); assertEquals(0, status); } + + @Test + public void testOfflineImageViewerMaxSizeAndStepOptions() throws Exception { + final ByteArrayOutputStream bytes = new ByteArrayOutputStream(); + final PrintStream out = new PrintStream(bytes); + final PrintStream oldOut = System.out; + try { + System.setOut(out); + // Add the -h option to make the test only for option parsing, + // and don't need to do the following operations. + OfflineImageViewer.main(new String[] {"-i", "-", "-o", "-", "-p", + "FileDistribution", "-maxSize", "512", "-step", "8", "-h"}); + Assert.assertFalse(bytes.toString().contains( + "Error parsing command-line options: ")); + } finally { + System.setOut(oldOut); + IOUtils.closeStream(out); + } + } } --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org