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 2017/10/04 15:18:15 UTC

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

Repository: hbase
Updated Branches:
  refs/heads/master 0a24178d0 -> 2f2513255


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

Signed-off-by: tedyu <yu...@gmail.com>


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

Branch: refs/heads/master
Commit: 2f2513255df0b3e46793c5e9284b9555f1bccd0a
Parents: 0a24178
Author: Amit <ak...@salesforce.com>
Authored: Wed Oct 4 14:54:35 2017 +0530
Committer: tedyu <yu...@gmail.com>
Committed: Wed Oct 4 08:18:08 2017 -0700

----------------------------------------------------------------------
 .../hadoop/hbase/backup/impl/IncrementalTableBackupClient.java  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/2f251325/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
----------------------------------------------------------------------
diff --git a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
index 4efe04b..52f6b5c 100644
--- a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
+++ b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
@@ -221,7 +221,7 @@ public class IncrementalTableBackupClient extends TableBackupClient {
       // fail the overall backup and return
       failBackup(conn, backupInfo, backupManager, e, "Unexpected Exception : ",
         BackupType.INCREMENTAL, conf);
-      return;
+      throw new IOException(e);
     }
 
     // case INCREMENTAL_COPY:
@@ -237,7 +237,7 @@ public class IncrementalTableBackupClient extends TableBackupClient {
       String msg = "Unexpected exception in incremental-backup: incremental copy " + backupId;
       // fail the overall backup and return
       failBackup(conn, backupInfo, backupManager, e, msg, BackupType.INCREMENTAL, conf);
-      return;
+      throw new IOException(e);
     }
     // case INCR_BACKUP_COMPLETE:
     // 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) {
       failBackup(conn, backupInfo, backupManager, e, "Unexpected Exception : ",
         BackupType.INCREMENTAL, conf);
+      throw new IOException(e);
     }
   }