Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-18932 Backup masking exception in a scenario and though it fails , it shows success message 

Signed-off-by: tedyu <yuzhihong@gmail.com>
This commit is contained in:
Amit 2017-10-04 14:54:35 +05:30 committed by tedyu
parent 35094bf4d5
commit e138ea6a59
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
View File

@ -221,7 +221,7 @@ public class IncrementalTableBackupClient extends TableBackupClient {
// fail the overall backup and return // fail the overall backup and return
failBackup(conn, backupInfo, backupManager, e, "Unexpected Exception : ", failBackup(conn, backupInfo, backupManager, e, "Unexpected Exception : ",
BackupType.INCREMENTAL, conf); BackupType.INCREMENTAL, conf);
return; throw new IOException(e);
} }
// case INCREMENTAL_COPY: // case INCREMENTAL_COPY:
@ -237,7 +237,7 @@ public class IncrementalTableBackupClient extends TableBackupClient {
String msg = "Unexpected exception in incremental-backup: incremental copy " + backupId; String msg = "Unexpected exception in incremental-backup: incremental copy " + backupId;
// fail the overall backup and return // fail the overall backup and return
failBackup(conn, backupInfo, backupManager, e, msg, BackupType.INCREMENTAL, conf); failBackup(conn, backupInfo, backupManager, e, msg, BackupType.INCREMENTAL, conf);
return; throw new IOException(e);
} }
// case INCR_BACKUP_COMPLETE: // case INCR_BACKUP_COMPLETE:
// set overall backup status: complete. Here we make sure to complete the backup. // set overall backup status: complete. Here we make sure to complete the backup.
@ -266,6 +266,7 @@ public class IncrementalTableBackupClient extends TableBackupClient {
} catch (IOException e) { } catch (IOException e) {
failBackup(conn, backupInfo, backupManager, e, "Unexpected Exception : ", failBackup(conn, backupInfo, backupManager, e, "Unexpected Exception : ",
BackupType.INCREMENTAL, conf); BackupType.INCREMENTAL, conf);
throw new IOException(e);
} }
} }