[HIVE-17255] hive_metastoreConstants.TABLE_IS_TRANSACTIONAL vs ConfVars.HIVE_TRANSACTIONAL_TABLE_SCAN

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: Transactions
• Target Version/s: 3.0.0
• Assignee: Unassigned
• Reporter: Eugene Koifman
• Created: 04/Aug/17 22:48
• Updated: 28/Jul/21 14:13

Description

constructor of Context() has
boolean isTableTransactional = conf.getBoolean(hive_metastoreConstants.TABLE_IS_TRANSACTIONAL, false).

This looks wrong. Everywhere else we use ConfVars.HIVE_TRANSACTIONAL_TABLE_SCAN.
(yet someone does set it - can't find where)

Utilities.copyTablePropertiesToConf() copies all table props to JobConf

There places in the code setting/expecting ConfVars.HIVE_TRANSACTIONAL_TABLE_SCAN and other places setting/expecting hive_metastoreConstants.TABLE_IS_TRANSACTIONAL. This is inconsistent.