You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2017/09/02 14:37:05 UTC

hbase git commit: HBASE-18741 Remove cancel command from backup code

Repository: hbase
Updated Branches:
  refs/heads/master e1eb53296 -> 7c51d3f2e


HBASE-18741 Remove cancel command from backup code

Signed-off-by: tedyu <yu...@gmail.com>


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

Branch: refs/heads/master
Commit: 7c51d3f2e1f85230db22d133f2422e7b3cd12559
Parents: e1eb532
Author: Peter Somogyi <ps...@cloudera.com>
Authored: Sat Sep 2 12:56:05 2017 +0200
Committer: tedyu <yu...@gmail.com>
Committed: Sat Sep 2 07:36:56 2017 -0700

----------------------------------------------------------------------
 .../hbase/backup/impl/BackupCommands.java       | 27 --------------------
 1 file changed, 27 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/7c51d3f2/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
----------------------------------------------------------------------
diff --git a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
index 5b91e34..187a014 100644
--- a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
+++ b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
@@ -106,9 +106,6 @@ public final class BackupCommands {
 
   public static final String REPAIR_CMD_USAGE = "Usage: hbase backup repair\n";
 
-  public static final String CANCEL_CMD_USAGE = "Usage: hbase backup cancel <backup_id>\n"
-      + "  backup_id       Backup image id\n";
-
   public static final String SET_CMD_USAGE = "Usage: hbase backup set COMMAND [name] [tables]\n"
       + "  name            Backup set name\n"
       + "  tables          Comma separated list of tables.\n" + "COMMAND is one of:\n"
@@ -221,9 +218,6 @@ public final class BackupCommands {
     case DELETE:
       cmd = new DeleteCommand(conf, cmdline);
       break;
-    case CANCEL:
-      cmd = new CancelCommand(conf, cmdline);
-      break;
     case HISTORY:
       cmd = new HistoryCommand(conf, cmdline);
       break;
@@ -427,8 +421,6 @@ public final class BackupCommands {
         System.out.println(PROGRESS_CMD_USAGE);
       } else if (BackupCommand.DELETE.name().equalsIgnoreCase(type)) {
         System.out.println(DELETE_CMD_USAGE);
-      } else if (BackupCommand.CANCEL.name().equalsIgnoreCase(type)) {
-        System.out.println(CANCEL_CMD_USAGE);
       } else if (BackupCommand.SET.name().equalsIgnoreCase(type)) {
         System.out.println(SET_CMD_USAGE);
       } else {
@@ -739,25 +731,6 @@ public final class BackupCommands {
     }
   }
 
-  // TODO Cancel command
-
-  private static class CancelCommand extends Command {
-
-    CancelCommand(Configuration conf, CommandLine cmdline) {
-      super(conf);
-      this.cmdline = cmdline;
-    }
-
-    @Override
-    public void execute() throws IOException {
-      throw new UnsupportedOperationException("Cancel command is not supported yet.");
-    }
-
-    @Override
-    protected void printUsage() {
-    }
-  }
-
   private static class HistoryCommand extends Command {
 
     private final static int DEFAULT_HISTORY_LENGTH = 10;