Skip to content

fix: disable native C2R for legacy Iceberg scans [iceberg]#3663

Merged
mbutrovich merged 2 commits intoapache:mainfrom
mbutrovich:disable_native_c2r_iceberg_java
Mar 11, 2026
Merged

fix: disable native C2R for legacy Iceberg scans [iceberg]#3663
mbutrovich merged 2 commits intoapache:mainfrom
mbutrovich:disable_native_c2r_iceberg_java

Conversation

@mbutrovich
Copy link
Contributor

@mbutrovich mbutrovich commented Mar 11, 2026

Which issue does this PR close?

Closes #.

Rationale for this change

#3657 (comment)

We're seeing issues with the legacy Iceberg codepath in the last day or two, and I suspect enabling native columnar to row by default.

What changes are included in this PR?

  • Set the config to false in CometScanRule.

How are these changes tested?

  • Existing tests that are currently failing on main.

@mbutrovich mbutrovich changed the title fix: Set COMET_NATIVE_COLUMNAR_TO_ROW_ENABLED to false for legacy Iceberg [iceberg[ fix: Set COMET_NATIVE_COLUMNAR_TO_ROW_ENABLED to false for legacy Iceberg [iceberg] Mar 11, 2026
@mbutrovich mbutrovich marked this pull request as ready for review March 11, 2026 14:10
@mbutrovich mbutrovich marked this pull request as draft March 11, 2026 14:28
@mbutrovich mbutrovich changed the title fix: Set COMET_NATIVE_COLUMNAR_TO_ROW_ENABLED to false for legacy Iceberg [iceberg] fix: disable native C2R for legacy Iceberg [iceberg] Mar 11, 2026
@mbutrovich mbutrovich changed the title fix: disable native C2R for legacy Iceberg [iceberg] fix: disable native C2R for legacy Iceberg scans [iceberg] Mar 11, 2026
@andygrove andygrove marked this pull request as ready for review March 11, 2026 16:49
@mbutrovich mbutrovich merged commit b9f29c8 into apache:main Mar 11, 2026
242 of 246 checks passed
@mbutrovich mbutrovich deleted the disable_native_c2r_iceberg_java branch March 11, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants