Skip to content

chore(deps): bump iceberg-rust to parallel file scanning fork (release/2.0) - #11330

Closed
phillipleblanc wants to merge 1031 commits into
release/2.0from
phillip/260615-iceberg-parallel-scan-rel20
Closed

chore(deps): bump iceberg-rust to parallel file scanning fork (release/2.0)#11330
phillipleblanc wants to merge 1031 commits into
release/2.0from
phillip/260615-iceberg-parallel-scan-rel20

Conversation

@phillipleblanc

Copy link
Copy Markdown
Contributor

release/2.0 backport of #11328. Re-pins spiceai/iceberg-rust to the eager task-bucketing parallel-scan change on the DF52 line.

Not a literal cherry-pick of #11328 — that pins the DF53 line (spiceai-0.9.1-df-53); this pins the DF52 line release/2.0 builds against. The spiceai-52 branch is renamed spiceai-0.9.1-df-52 per the spiceai-<iceberg>-df-<datafusion> convention (alias retained); the version requirement stays caret "0.9.0" (satisfied by the fork's 0.9.1).

Effect on Spice

IcebergTableProvider::scan() now plans files eagerly and distributes file scan tasks across min(target_partitions, n_files) DataFusion partitions, so Iceberg file reads are scheduled concurrently. No Spice code changes — Spice consumes the provider only via IcebergTableProvider::try_new; the fork preserves Spice's Iceberg limit-pushdown.

Validation

  • cargo check -p data_components clean against the bumped rev (Spice DataFusion 52 fork).
  • Fork validated independently: 86 lib tests + all 9 df_* sqllogictest schedules + integration test; clippy/fmt clean (DataFusion 52.2).

Note

Draft pending merge of spiceai/iceberg-rust#44. Pins the fork branch head; re-pin to the merge commit once #44 merges into spiceai-0.9.1-df-52, then mark ready.

lukekim and others added 30 commits March 3, 2026 15:54
Merge OSS upstream 'trunk' Mar 3, 2026
* fix: update Dockerfile to conditionally install NFS support

* fix: add libclang-dev to Dockerfile dependencies

* fix: derive NFS install from CARGO_FEATURES instead of separate INSTALL_NFS arg

Address review feedback: Remove the INSTALL_NFS build arg that could
drift out of sync with CARGO_FEATURES. Instead, derive the libnfs-dev
install from CARGO_FEATURES using the same grep pattern as ODBC.

Also note: NFS and SMB features are intentionally removed from default,
data, and allocator variants and moved to a dedicated 'nas' variant.
This is a deliberate change to reduce image size for variants that
don't need NAS support.
* Fix Saffron benchmarks dispatch

* Trigger CI

* Trigger CI
…03-04

# Conflicts:
#	tools/testoperator/dispatch/tpch/sf1/federated/iceberg[catalog].yaml
Install odbc in `install-spice` action
fix: Align CUDA asset names in Dockerfile and install tests with build output
Merges spiceai/spiceai trunk as of 2026-03-09 into spicehq/spiceai trunk.
Includes runner name replacements and enterprise version preservation.
* Enterprise release endgame: update list of test app

* Update end_game_enterprise.md
lukekim and others added 19 commits May 27, 2026 23:56
…-oss-release

Prepare v2.0.0-rc.5-enterprise
Include region in `spice dataset configure` for spice.ai command
* Refactor Clickhouse installation in CI workflow

Updated Clickhouse installation steps for Linux and macOS with specific versioning and repository setup.

* update cargo aes

---------

Co-authored-by: Jeadie <jeadie@users.noreply.github.com>
requires `-helm` suffix.
Update version format in Chart.yaml
Co-authored-by: Spice Schema Bot <schema-bot@spice.ai>
…e/2.0)

Re-pins iceberg-rust to spiceai-0.9.1-df-52 (rev 65463fe6), picking up
the eager task-bucketing parallel-scan change (fork PR
spiceai/iceberg-rust#44, port of apache/iceberg-rust#2298).

release/2.0 counterpart of the trunk change in #11328. The DF52 fork line
spiceai-52 is renamed spiceai-0.9.1-df-52 per the spiceai-<iceberg>-df-<df>
convention (alias retained; version req stays caret 0.9.0, satisfied by
0.9.1). No Spice code changes — only IcebergTableProvider::try_new is used.
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Pull with Spice Failed

🏷️ Auto-applied labels:

  • area/docs
  • area/ci
  • area/tests
  • area/config
  • kind/dependencies
  • size/xl
  • kind/chore

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ No banned labels detected
  • ✅ Has a label from required category kind/
  • ✅ Has a label from required category area/

Failed checks:

  • ❌ At least one assignee is required for this pull request.

Please address these issues and update your pull request.

@phillipleblanc

Copy link
Copy Markdown
Contributor Author

Closing — this branch was accidentally based on a stale local release/2.0 bookmark (ancient commit c8c7fa8) instead of the current origin/release/2.0 (d16271c), so the diff is bogus. Re-opening off the correct base.

@phillipleblanc
phillipleblanc deleted the phillip/260615-iceberg-parallel-scan-rel20 branch June 15, 2026 03:17
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.

7 participants