[HIVE-24516] Txnhandler onrename might ignore exceptions

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: Hive
• Labels: pull-request-available
• Assignee: Peter Varga
• Reporter: Peter Varga
• Created: 10/Dec/20 09:44
• Updated: 18/Feb/21 00:52

Description

This is a followup on HIVE-24193. Table not exists errors shouldn't be ignored in the first place.

} catch (SQLException e) {
        LOG.debug("Going to rollback: " + callSig);
        rollbackDBConn(dbConn);
        checkRetryable(dbConn, e, callSig);
        if (e.getMessage().contains("does not exist")) {
          LOG.warn("Cannot perform " + callSig + " since metastore table does not exist");
        } else {
          throw new MetaException("Unable to " + callSig + ":" + StringUtils.stringifyException(e));
        }
      }

This error handling might have been put there for backard compatibility for missing acid metadata tables, but this is not needed anymore.

Issue Links

• links to: GitHub Pull Request #1762