Skip to content

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

Open
deniskuzZ wants to merge 1 commit intoapache:masterfrom
deniskuzZ:HIVE-29530
Open

HIVE-29530: Hive's explicit transaction doesn't commit#6395
deniskuzZ wants to merge 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

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