[HIVE-9436] RetryingMetaStoreClient does not retry JDOExceptions

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.13.1, 0.14.0
• Fix Version/s: 1.0.0
• Assignee: Sushanth Sowmyan
• Reporter: Sushanth Sowmyan
• Created: 21/Jan/15 21:09
• Updated: 05/Jan/17 18:02
• Resolved: 28/Jan/15 23:58

Description

RetryingMetaStoreClient has a bug in the following bit of code:

        } else if ((e.getCause() instanceof MetaException) &&
            e.getCause().getMessage().matches("JDO[a-zA-Z]*Exception")) {
          caughtException = (MetaException) e.getCause();
        } else {
          throw e.getCause();
        }

The bug here is that java String.matches matches the entire string to the regex, and thus, that match will fail if the message contains anything before or after JDO[a-zA-Z]*Exception. The solution, however, is very simple, we should match (?s).*JDO[a-zA-Z]*Exception.*

Issue Links

• relates to: HIVE-10384 RetryingMetaStoreClient does not retry wrapped TTransportExceptions