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 in...@apache.org on 2018/07/25 01:32:06 UTC

[23/50] hadoop git commit: HDDS-275. Add message output for succeeded -deleteVolume CLI. Contributed by Nilotpal Nandi.

HDDS-275. Add message output for succeeded -deleteVolume CLI. Contributed by Nilotpal Nandi.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/c7ae5567
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/c7ae5567
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/c7ae5567

Branch: refs/heads/HADOOP-15461
Commit: c7ae55675ed56cb18266425c02674a5a87561e0c
Parents: 6837121
Author: Xiaoyu Yao <xy...@apache.org>
Authored: Fri Jul 20 13:12:07 2018 -0700
Committer: Xiaoyu Yao <xy...@apache.org>
Committed: Fri Jul 20 13:12:53 2018 -0700

----------------------------------------------------------------------
 .../test/java/org/apache/hadoop/ozone/ozShell/TestOzoneShell.java  | 2 ++
 .../hadoop/ozone/web/ozShell/volume/DeleteVolumeHandler.java       | 1 +
 2 files changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/c7ae5567/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/ozShell/TestOzoneShell.java
----------------------------------------------------------------------
diff --git a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/ozShell/TestOzoneShell.java b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/ozShell/TestOzoneShell.java
index 8f53049..573f097 100644
--- a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/ozShell/TestOzoneShell.java
+++ b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/ozShell/TestOzoneShell.java
@@ -249,6 +249,8 @@ public class TestOzoneShell {
     String[] args = new String[] {"-deleteVolume", url + "/" + volumeName,
         "-root"};
     assertEquals(0, ToolRunner.run(shell, args));
+    String output = out.toString();
+    assertTrue(output.contains("Volume " + volumeName + " is deleted"));
 
     // verify if volume has been deleted
     try {

http://git-wip-us.apache.org/repos/asf/hadoop/blob/c7ae5567/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/web/ozShell/volume/DeleteVolumeHandler.java
----------------------------------------------------------------------
diff --git a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/web/ozShell/volume/DeleteVolumeHandler.java b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/web/ozShell/volume/DeleteVolumeHandler.java
index d6facf6..2df788a 100644
--- a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/web/ozShell/volume/DeleteVolumeHandler.java
+++ b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/web/ozShell/volume/DeleteVolumeHandler.java
@@ -67,5 +67,6 @@ public class DeleteVolumeHandler extends Handler {
     }
 
     client.getObjectStore().deleteVolume(volumeName);
+    System.out.printf("Volume %s is deleted%n", volumeName);
   }
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org