From 747b5cb124d2a22c315d343ee81dd8422ad74e08 Mon Sep 17 00:00:00 2001 From: Michael Stack Date: Mon, 12 Dec 2011 19:24:28 +0000 Subject: [PATCH] HBASE-4881 Unhealthy region is on service caused by rollback of region splitting git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1213383 13f79535-47bb-0310-9956-ffa450edef68 --- .../hbase/regionserver/SplitTransaction.java | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java b/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java index f2d8bbc17dd..a214f4918ab 100644 --- a/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java +++ b/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java @@ -723,12 +723,18 @@ public class SplitTransaction { break; case CLOSED_PARENT_REGION: - // So, this returns a seqid but if we just closed and then reopened, we - // should be ok. On close, we flushed using sequenceid obtained from - // hosting regionserver so no need to propagate the sequenceid returned - // out of initialize below up into regionserver as we normally do. - // TODO: Verify. - this.parent.initialize(); + try { + // So, this returns a seqid but if we just closed and then reopened, we + // should be ok. On close, we flushed using sequenceid obtained from + // hosting regionserver so no need to propagate the sequenceid returned + // out of initialize below up into regionserver as we normally do. + // TODO: Verify. + this.parent.initialize(); + } catch (IOException e) { + LOG.error("Failed rollbacking CLOSED_PARENT_REGION of region " + + this.parent.getRegionNameAsString(), e); + throw new RuntimeException(e); + } break; case STARTED_REGION_A_CREATION: