[
https://issues.apache.org/jira/browse/HBASE-5974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13278491#comment-13278491
]
Jieshan Bean commented on HBASE-5974:
-------------------------------------
bq.When a RemoteException comes, can we mark that as DoNotRetryIOException?
I think it should be marked as DNRI.
bq.Now in normal close() of scanner from client side and lease timeout based
close, we call CP hooks. But what abt the scanner getting removed from RS when
the NSRE happens.
So what's your suggestion, Anoop? call CP hooks in the finally section?
The patch looks nice. A minor comment:
{noformat}
RegionScanner scanner = scanners.get(scannerIdString).s;
{noformat}
Is there one chance of "scanners.get(scannerIdString)" is null? any way, the
direct access to that private variable is not good.
> Scanner retry behavior with RPC timeout on next() seems incorrect
> -----------------------------------------------------------------
>
> Key: HBASE-5974
> URL: https://issues.apache.org/jira/browse/HBASE-5974
> Project: HBase
> Issue Type: Bug
> Components: client, regionserver
> Affects Versions: 0.90.7, 0.92.1, 0.94.0, 0.96.0
> Reporter: Todd Lipcon
> Priority: Critical
> Attachments: HBASE-5974_0.94.patch
>
>
> I'm seeing the following behavior:
> - set RPC timeout to a short value
> - call next() for some batch of rows, big enough so the client times out
> before the result is returned
> - the HConnectionManager stuff will retry the next() call to the same server.
> At this point, one of two things can happen: 1) the previous next() call will
> still be processing, in which case you get a LeaseException, because it was
> removed from the map during the processing, or 2) the next() call will
> succeed but skip the prior batch of rows.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
