Skip to content

HIVE-29436: Iceberg materialized view rebuild locking issue#6330

Open
kasakrisz wants to merge 1 commit intoapache:masterfrom
kasakrisz:HIVE-29436-master-iceberg-mv-lock
Open

HIVE-29436: Iceberg materialized view rebuild locking issue#6330
kasakrisz wants to merge 1 commit intoapache:masterfrom
kasakrisz:HIVE-29436-master-iceberg-mv-lock

Conversation

@kasakrisz
Copy link
Contributor

@kasakrisz kasakrisz commented Feb 20, 2026

What changes were proposed in this pull request?

Do not acquire a rebuild lock for materialized views stored by Iceberg because we rely on Iceberg's concurrency management.

Why are the changes needed?

Subsequent materialized view rebuilds may fail. See jira for details: https://issues.apache.org/jira/browse/HIVE-29436

Does this PR introduce any user-facing change?

No.

How was this patch tested?

mvn test -Dtest.output.overwrite -Dtest=TestIcebergCliDriver.java -Dqfile=mv_iceberg_orc2.q -pl itests/qtest-iceberg -Pitests

@kasakrisz kasakrisz marked this pull request as draft February 20, 2026 15:31
@sonarqubecloud
Copy link

@kasakrisz kasakrisz requested a review from deniskuzZ March 5, 2026 13:44
@kasakrisz kasakrisz marked this pull request as ready for review March 5, 2026 13:44
alter materialized view mat1 rebuild;

alter materialized view mat1 rebuild;
alter materialized view mat2 rebuild;

Choose a reason for hiding this comment

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

In this test we are checking if rebuild for different materialized views succeeds. Can we also test if rebuilds of the same MV also works both in a serialized way within a session and in a concurrent sessions scenario ?

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