You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ch...@apache.org on 2017/09/04 18:17:57 UTC

hbase git commit: HBASE-18746 Throw exception with job.getStatus().getFailureInfo() when ExportSnapshot fails

Repository: hbase
Updated Branches:
  refs/heads/master 32ddb1f60 -> bea5de02b


HBASE-18746 Throw exception with job.getStatus().getFailureInfo() when ExportSnapshot fails

Signed-off-by: Chia-Ping Tsai <ch...@gmail.com>


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

Branch: refs/heads/master
Commit: bea5de02b3c353913b6a3de2d5ec0507b901dbdf
Parents: 32ddb1f
Author: Chun-Hao Tang <ta...@gmail.com>
Authored: Mon Sep 4 02:58:34 2017 +0800
Committer: Chia-Ping Tsai <ch...@gmail.com>
Committed: Tue Sep 5 01:54:29 2017 +0800

----------------------------------------------------------------------
 .../java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java    | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/bea5de02/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java
----------------------------------------------------------------------
diff --git a/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java b/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java
index e80410f..8b5af2b 100644
--- a/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java
+++ b/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java
@@ -826,9 +826,7 @@ public class ExportSnapshot extends AbstractHBaseTool implements Tool {
 
     // Run the MR Job
     if (!job.waitForCompletion(true)) {
-      // TODO: Replace the fixed string with job.getStatus().getFailureInfo()
-      // when it will be available on all the supported versions.
-      throw new ExportSnapshotException("Copy Files Map-Reduce Job failed");
+      throw new ExportSnapshotException(job.getStatus().getFailureInfo());
     }
   }