You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2018/03/07 10:14:49 UTC

[13/33] hbase git commit: HBASE-20121 Fix findbugs warning for RestoreTablesClient

HBASE-20121 Fix findbugs warning for RestoreTablesClient


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

Branch: refs/heads/HBASE-19064
Commit: f89a1f7d7ac0bdc7bc5af8334f55461ba07bc853
Parents: 4a4c012
Author: tedyu <yu...@gmail.com>
Authored: Mon Mar 5 19:36:19 2018 -0800
Committer: tedyu <yu...@gmail.com>
Committed: Mon Mar 5 19:36:19 2018 -0800

----------------------------------------------------------------------
 .../apache/hadoop/hbase/backup/impl/RestoreTablesClient.java  | 7 -------
 1 file changed, 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/f89a1f7d/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java
----------------------------------------------------------------------
diff --git a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java
index c52d658..80f0d60 100644
--- a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java
+++ b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java
@@ -23,9 +23,7 @@ import static org.apache.hadoop.hbase.backup.BackupRestoreConstants.JOB_NAME_CON
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.HashMap;
-import java.util.HashSet;
 import java.util.List;
-import java.util.Set;
 import java.util.TreeSet;
 
 import org.apache.commons.lang3.StringUtils;
@@ -206,7 +204,6 @@ public class RestoreTablesClient {
   private void restore(HashMap<TableName, BackupManifest> backupManifestMap,
       TableName[] sTableArray, TableName[] tTableArray, boolean isOverwrite) throws IOException {
     TreeSet<BackupImage> restoreImageSet = new TreeSet<>();
-    Set<String> backupIdSet = new HashSet<>();
 
     for (int i = 0; i < sTableArray.length; i++) {
       TableName table = sTableArray[i];
@@ -229,10 +226,6 @@ public class RestoreTablesClient {
           LOG.info("Backup: " + image.getBackupId() + " "
               + HBackupFileSystem.getTableBackupDir(image.getRootDir(), image.getBackupId(),
                   table));
-          if (image.getType() == BackupType.INCREMENTAL) {
-            backupIdSet.add(image.getBackupId());
-            LOG.debug("adding " + image.getBackupId() + " for bulk load");
-          }
         }
       }
     }