Skip to content

HIVE-29530: Hive's explicit transaction doesn't commit#6395

Merged
deniskuzZ merged 1 commit intoapache:masterfrom
deniskuzZ:HIVE-29530
Mar 31, 2026
Merged

HIVE-29530: Hive's explicit transaction doesn't commit#6395
deniskuzZ merged 1 commit intoapache:masterfrom
deniskuzZ:HIVE-29530

Conversation

@deniskuzZ
Copy link
Copy Markdown
Member

@deniskuzZ deniskuzZ commented Mar 30, 2026

What changes were proposed in this pull request?

  1. Hive's explicit transaction doesn't commit;
  2. Removed assume check in isImplicitTxn cause multi-insert could have >1 num statements;
  3. Fixed the DriverTxnHandler.destroy

Why are the changes needed?

BugFix

Does this PR introduce any user-facing change?

No

How was this patch tested?

mvn test -Dtest=TestIcebergCliDriver -Dqfile=iceberg_multi_table_txn.q

Copy link
Copy Markdown
Contributor

@kasakrisz kasakrisz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sonarqubecloud
Copy link
Copy Markdown

@deniskuzZ deniskuzZ merged commit 291cfd1 into apache:master Mar 31, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants