Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Fri, 27 Nov 2015 15:46:10 +0000 (UTC)
From: "Ted Yu (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.12916661.1448639084000.210203.1448639170947@Atlassian.JIRA>
In-Reply-To: <JIRA.12916661.1448639084000@Atlassian.JIRA>
References: <JIRA.12916661.1448639084000@Atlassian.JIRA>
 <JIRA.12916661.1448639084999@arcas>
Subject: [jira] [Updated] (HBASE-14893) Race between mutation on region and
 region closing operation leads to NotServingRegionException
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/HBASE-14893?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-14893:
---------------------------
    Status: Patch Available  (was: Open)

> Race between mutation on region and region closing operation leads to NotServingRegionException
> -----------------------------------------------------------------------------------------------
>
>                 Key: HBASE-14893
>                 URL: https://issues.apache.org/jira/browse/HBASE-14893
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: 14893-v1.txt
>
>
> During system test involving Phoenix local index, we observed the following in region server log:
> {code}
> 2015-11-25 08:20:03,258 DEBUG [B.defaultRpcServer.handler=38,queue=2,port=49524] ipc.RpcServer: B.defaultRpcServer.handler=38,queue=2,port=49524: callId: 28 service:             ClientService methodName: Scan size: 277 connection: 100.75.224.9:64138^M
> org.apache.hadoop.hbase.NotServingRegionException: GIGANTIC_TABLE,,1448439565197.0dc568cba621f11fd848ef87241d8535. is closing^M
>   at org.apache.hadoop.hbase.regionserver.HRegion.startRegionOperation(HRegion.java:7649)^M
>   at org.apache.hadoop.hbase.regionserver.HRegion.batchMutate(HRegion.java:2803)^M
>   at org.apache.hadoop.hbase.regionserver.HRegion.batchMutate(HRegion.java:2760)^M
>   at org.apache.phoenix.coprocessor.UngroupedAggregateRegionObserver.commitBatch(UngroupedAggregateRegionObserver.java:140)^M
>   at org.apache.phoenix.coprocessor.UngroupedAggregateRegionObserver.doPostScannerOpen(UngroupedAggregateRegionObserver.java:417)^M
>   at org.apache.phoenix.coprocessor.BaseScannerRegionObserver.postScannerOpen(BaseScannerRegionObserver.java:177)^M
>   at org.apache.hadoop.hbase.regionserver.RegionCoprocessorHost$52.call(RegionCoprocessorHost.java:1318)^M
>   at org.apache.hadoop.hbase.regionserver.RegionCoprocessorHost$RegionOperation.call(RegionCoprocessorHost.java:1673)^M
>   at org.apache.hadoop.hbase.regionserver.RegionCoprocessorHost.execOperation(RegionCoprocessorHost.java:1748)^M
>   at org.apache.hadoop.hbase.regionserver.RegionCoprocessorHost.execOperationWithResult(RegionCoprocessorHost.java:1712)^M
>   at org.apache.hadoop.hbase.regionserver.RegionCoprocessorHost.postScannerOpen(RegionCoprocessorHost.java:1313)^M
>   at org.apache.hadoop.hbase.regionserver.RSRpcServices.scan(RSRpcServices.java:2259)^M
>   at org.apache.hadoop.hbase.protobuf.generated.ClientProtos$ClientService$2.callBlockingMethod(ClientProtos.java:32205)^M
>   at org.apache.hadoop.hbase.ipc.RpcServer.call(RpcServer.java:2114)^M
> {code}
> Here is related code snippet from UngroupedAggregateRegionObserver:
> {code}
>         region.startRegionOperation();
>         try {
> ...
>                             // Commit in batches based on UPSERT_BATCH_SIZE_ATTRIB in config
>                             if (!indexMutations.isEmpty() && batchSize > 0 &&
>                                     indexMutations.size() % batchSize == 0) {
>                             	commitBatch(region, indexMutations, null);
> {code}
> In startRegionOperation(), read lock on region was obtained. So region close should not proceed until the operation completes.
> However, we still got region closing because region#closing is set to true before write lock is taken in region#doClose() :
> {code}
>     this.closing.set(true);
>     status.setStatus("Disabling writes for close");
>     // block waiting for the lock for closing
>     lock.writeLock().lock();
> {code}
> Proposed fix is to obtain write lock first.
> Thanks to Rajeshbabu for offline discussion.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)