[ 
https://issues.apache.org/jira/browse/PHOENIX-2903?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15295795#comment-15295795
 ] 

James Taylor commented on PHOENIX-2903:
---------------------------------------

Thanks, [~rajeshbabu]. So do we not need this change and if not, where will it 
be handled?
{noformat}
                 region.startRegionOperation();
                 acquiredLock = true;
                 synchronized (scanner) {
+                    // Check again while we have the region lock as it's 
possible
+                    // we've split after the check we did in preScannerOpen()
+                    throwIfScanOutOfRegion(scan, 
c.getEnvironment().getRegion());
{noformat}

I'll get the rest of the patch over soon.


> Handle split during scan for row key ordered aggregations
> ---------------------------------------------------------
>
>                 Key: PHOENIX-2903
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-2903
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: James Taylor
>            Assignee: James Taylor
>         Attachments: PHOENIX-2903_v1.patch, PHOENIX-2903_wip.patch
>
>
> Currently a hole in our split detection code



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

Reply via email to