[HIVE-26386] Exceptions thrown in ObjectStore should contain all causes

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 4.0.0-alpha-1
• Component/s: Hive
• Labels: pull-request-available
• Target Version/s: 4.0.0-alpha-2
• Assignee: Wechar
• Reporter: Wechar
• Created: 12/Jul/22 18:21
• Updated: 06/Dec/23 10:37

Description

RetryingHMSHandler will retry the failed operations if the exception is caused by JDOException or NucleusException, the logic is in RetryingHMSHandler.java#L185:

 } else if (e.getCause() instanceof MetaException && e.getCause().getCause() != null) {
          if (e.getCause().getCause() instanceof javax.jdo.JDOException ||
              e.getCause().getCause() instanceof NucleusException) {
            // The JDOException or the Nucleus Exception may be wrapped further in a MetaException
            caughtException = e.getCause().getCause();

But in ObjectStore.java, we found some exceptions thrown as MetaException are not wrapped by the causes, which cause some operations failed by JDOException can not do retry.

Issue Links

• links to: GitHub Pull Request #3431