Andrew Kyle Purtell created PHOENIX-7895:
--------------------------------------------

             Summary: Bound stale region retries in BaseResultIterators
                 Key: PHOENIX-7895
                 URL: https://issues.apache.org/jira/browse/PHOENIX-7895
             Project: Phoenix
          Issue Type: Sub-task
    Affects Versions: 5.3.1, 5.4.0
            Reporter: Andrew Kyle Purtell
            Assignee: Andrew Kyle Purtell


In {{BaseResultIterators.getIterators}} we should always check {{retryCount} 
before callling {{recreateIterators}}, so we avoid potential infinite recursion 
between {{getIterators}} and {{recreateIterators}}. Something like:

{noformat}
if (retryCount <= 0) {
  throw e2;
}
concatIterators = recreateIterators(services, isLocalIndex, allIterators, 
iterators,
    isReverse, maxQueryEndTime, previousScan, clearedCache, concatIterators,
    scanPairItr, scanPair, retryCount - 1);
{noformat}

The existing {{retryCount}} check when handling 
{{HashJoinCacheNotFoundException}} specifically becomes redundant and can be 
removed.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to