[HBASE-22047] LeaseException in Scan should be retired

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.0, 2.0.4, 2.1.3
• Fix Version/s: 3.0.0-alpha-1, 1.5.0, 2.2.0, 2.3.0, 2.1.5
• Component/s: Client, Scanners
• Hadoop Flags: Reviewed
• Assignee: Igor Rudenko
• Reporter: Allan Yang
• Created: 13/Mar/19 06:50
• Updated: 02/Aug/19 12:07
• Resolved: 05/May/19 03:35

Description

We should retry LeaseException just like other exceptions like OutOfOrderScannerNextException and UnknownScannerException
Code in ClientScanner：

if ((cause != null && cause instanceof NotServingRegionException) ||
        (cause != null && cause instanceof RegionServerStoppedException) ||
        e instanceof OutOfOrderScannerNextException || e instanceof UnknownScannerException ||
        e instanceof ScannerResetException) {
      // Pass. It is easier writing the if loop test as list of what is allowed rather than
      // as a list of what is not allowed... so if in here, it means we do not throw.
      if (retriesLeft <= 0) {
        throw e; // no more retries
      }

Issue Links

• blocks: HBASE-22017 Master Fails to become active due to the data race bug in region server
• links to: GitHub Pull Request #158