HBASE-20121 Fix findbugs warning for RestoreTablesClient
This commit is contained in:
parent
4a4c012049
commit
f89a1f7d7a
|
@ -23,9 +23,7 @@ import static org.apache.hadoop.hbase.backup.BackupRestoreConstants.JOB_NAME_CON
|
||||||
import java.io.IOException;
|
import java.io.IOException;
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
import java.util.HashMap;
|
import java.util.HashMap;
|
||||||
import java.util.HashSet;
|
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
import java.util.Set;
|
|
||||||
import java.util.TreeSet;
|
import java.util.TreeSet;
|
||||||
|
|
||||||
import org.apache.commons.lang3.StringUtils;
|
import org.apache.commons.lang3.StringUtils;
|
||||||
|
@ -206,7 +204,6 @@ public class RestoreTablesClient {
|
||||||
private void restore(HashMap<TableName, BackupManifest> backupManifestMap,
|
private void restore(HashMap<TableName, BackupManifest> backupManifestMap,
|
||||||
TableName[] sTableArray, TableName[] tTableArray, boolean isOverwrite) throws IOException {
|
TableName[] sTableArray, TableName[] tTableArray, boolean isOverwrite) throws IOException {
|
||||||
TreeSet<BackupImage> restoreImageSet = new TreeSet<>();
|
TreeSet<BackupImage> restoreImageSet = new TreeSet<>();
|
||||||
Set<String> backupIdSet = new HashSet<>();
|
|
||||||
|
|
||||||
for (int i = 0; i < sTableArray.length; i++) {
|
for (int i = 0; i < sTableArray.length; i++) {
|
||||||
TableName table = sTableArray[i];
|
TableName table = sTableArray[i];
|
||||||
|
@ -229,10 +226,6 @@ public class RestoreTablesClient {
|
||||||
LOG.info("Backup: " + image.getBackupId() + " "
|
LOG.info("Backup: " + image.getBackupId() + " "
|
||||||
+ HBackupFileSystem.getTableBackupDir(image.getRootDir(), image.getBackupId(),
|
+ HBackupFileSystem.getTableBackupDir(image.getRootDir(), image.getBackupId(),
|
||||||
table));
|
table));
|
||||||
if (image.getType() == BackupType.INCREMENTAL) {
|
|
||||||
backupIdSet.add(image.getBackupId());
|
|
||||||
LOG.debug("adding " + image.getBackupId() + " for bulk load");
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue