[HIVE-26026] Use the new "REFUSED" compaction state where it makes sense

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.0.0-alpha-2
• Component/s: Hive
• Labels: pull-request-available
• Assignee: László Végh
• Reporter: László Végh
• Created: 11/Mar/22 13:58
• Updated: 16/Nov/22 13:50
• Resolved: 13/May/22 10:44

Description

The org.apache.hadoop.hive.ql.txn.compactor.Worker#findNextCompactionAndExecute method does several checks (The table/partition exists, is not sorted, there are enough files to compact, etc.) before it actually executes the compaction request. If the compaction request fails on any of these checks, it is put to "SUCCEEDED" state which is often misleading for users. SHOW COMPACTIONS will show these requests as succeeded without an error, while the table is not compacted at all.

For these cases, the state should be "REFUSED" instead of "SUCCEEDED" among with the appropriate error message.

Issue Links

• depends upon: HIVE-25915 Query based MINOR compaction fails with NPE if the data is loaded into the ACID table
• links to: GitHub Pull Request #3126