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 2018/03/06 03:36:27 UTC
hbase git commit: HBASE-20121 Fix findbugs warning for
RestoreTablesClient
Repository: hbase
Updated Branches:
refs/heads/master 4a4c01204 -> f89a1f7d7
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/master
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");
- }
}
}
}