You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2015/12/08 04:00:28 UTC
hbase git commit: HBASE-14893 Race between mutation on region and
region closing operation leads to NotServingRegionException
Repository: hbase
Updated Branches:
refs/heads/0.98 7899c7239 -> e2b70c3a3
HBASE-14893 Race between mutation on region and region closing operation leads to NotServingRegionException
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/e2b70c3a
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e2b70c3a
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e2b70c3a
Branch: refs/heads/0.98
Commit: e2b70c3a3cad0cada60eff67b58d9b1210291a26
Parents: 7899c72
Author: tedyu <yu...@gmail.com>
Authored: Fri Nov 27 15:32:42 2015 -0800
Committer: Andrew Purtell <ap...@apache.org>
Committed: Mon Dec 7 15:46:41 2015 -0800
----------------------------------------------------------------------
.../main/java/org/apache/hadoop/hbase/regionserver/HRegion.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/e2b70c3a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
index d3f2144..93cc064 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
@@ -1178,10 +1178,10 @@ public class HRegion implements HeapSize { // , Writable{
}
}
- this.closing.set(true);
- status.setStatus("Disabling writes for close");
// block waiting for the lock for closing
lock.writeLock().lock();
+ this.closing.set(true);
+ status.setStatus("Disabling writes for close");
try {
if (this.isClosed()) {
status.abort("Already got closed by another process");