[HIVE-27989] Wrong database name in MetaException from MetastoreDefaultTransformer.java

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 4.0.0
• Labels: pull-request-available
• Assignee: Butao Zhang
• Reporter: Riza Suminto
• Created: 08/Jan/24 20:21
• Updated: 29/Mar/24 17:49
• Resolved: 12/Jan/24 21:21

Description

There are two places in MetastoreDefaultTransformer.java where MetaException message use table.getTableName() instead of table.getDbName(), causing some confusion.

https://github.com/apache/hive/blob/24fffdc508f9402ad7145b59b50de738b27c92b4/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetastoreDefaultTransformer.java#L775

          "Database " + oldTable.getTableName() + " for table " + oldTable.getTableName() + " could not be found"); 

https://github.com/apache/hive/blob/24fffdc508f9402ad7145b59b50de738b27c92b4/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetastoreDefaultTransformer.java#L898

      throw new MetaException("Database " + table.getTableName() + " for table " + table.getTableName() + " could not be found"); 

This is revealed through IMPALA-12655.

Issue Links

• Discovered while testing: IMPALA-12655 PlannerTest.testProcessingCost seems flaky
• links to: GitHub Pull Request #4989