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 2016/06/03 16:31:14 UTC

hbase git commit: HBASE-15953 HBase backup set list command does not list any existing sets (Vladimir)

Repository: hbase
Updated Branches:
  refs/heads/HBASE-7912 c51700180 -> 964c0841d


HBASE-15953 HBase backup set list command does not list any existing sets (Vladimir)


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

Branch: refs/heads/HBASE-7912
Commit: 964c0841db67464fc5b96807dfd321b44de8b5d7
Parents: c517001
Author: tedyu <yu...@gmail.com>
Authored: Fri Jun 3 09:31:13 2016 -0700
Committer: tedyu <yu...@gmail.com>
Committed: Fri Jun 3 09:31:13 2016 -0700

----------------------------------------------------------------------
 .../org/apache/hadoop/hbase/backup/impl/BackupCommands.java | 5 ++++-
 .../java/org/apache/hadoop/hbase/backup/TestBackupBase.java | 9 ---------
 2 files changed, 4 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/964c0841/hbase-client/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
----------------------------------------------------------------------
diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
index 1fce171..9a8697a 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/backup/impl/BackupCommands.java
@@ -466,7 +466,10 @@ public final class BackupCommands {
       Configuration conf = getConf() != null? getConf(): HBaseConfiguration.create();
       try(final Connection conn = ConnectionFactory.createConnection(conf); 
           final BackupAdmin admin = conn.getAdmin().getBackupAdmin();){
-        admin.listBackupSets();
+        List<BackupSet> list = admin.listBackupSets();
+        for(BackupSet bs: list){
+          System.out.println(bs);
+        }
       }
     }
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/964c0841/hbase-server/src/test/java/org/apache/hadoop/hbase/backup/TestBackupBase.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/backup/TestBackupBase.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/backup/TestBackupBase.java
index 9683e16..31cf65a 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/backup/TestBackupBase.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/backup/TestBackupBase.java
@@ -249,15 +249,6 @@ public class TestBackupBase {
     }
   }
 
-//  protected BackupClient getBackupClient(){
-//    return BackupRestoreClientFactory.getBackupClient(conf1);
-//  }
-  
-//  protected RestoreClient getRestoreClient()
-//  {
-//    return BackupRestoreClientFactory.getRestoreClient(conf1);
-//  }
-
   protected BackupAdmin getBackupAdmin() throws IOException {
     return TEST_UTIL.getAdmin().getBackupAdmin();
   }