Skip to content

chore: upgrade itertools to 0.14#7424

Merged
wjones127 merged 1 commit into
lance-format:mainfrom
wjones127:chore/upgrade-itertools-rand
Jun 23, 2026
Merged

chore: upgrade itertools to 0.14#7424
wjones127 merged 1 commit into
lance-format:mainfrom
wjones127:chore/upgrade-itertools-rand

Conversation

@wjones127

Copy link
Copy Markdown
Contributor

DataFusion, object_store, and prost-derive already resolved to itertools 0.14, while our workspace pin held a second 0.13 copy in the dependency graph. Bumping the pin to 0.14 collapses the duplicate; only criterion retains 0.13, and that is dev/bench-only so it does not affect shipped crates.

No source changes were needed — we use no items removed between 0.13 and 0.14.

Verified with cargo check --workspace --tests --benches and cargo clippy --all --tests --benches -- -D warnings.


Context: this came out of a dependency-graph audit. A rand 0.9 → 0.10 bump was also considered, but dropped — rand 0.9 is held alive by DataFusion (and goosefs-sdk/hf-hub/jsonb), so bumping our code to 0.10 would not eliminate the duplicate, only move which copy our code uses. Worth revisiting once DataFusion moves to rand 0.10.

@github-actions github-actions Bot added A-deps Dependency updates chore labels Jun 23, 2026
@wjones127
wjones127 force-pushed the chore/upgrade-itertools-rand branch from 91f77ee to b25d8b2 Compare June 23, 2026 16:53
@github-actions github-actions Bot added the A-python Python bindings label Jun 23, 2026
DataFusion, object_store, and prost-derive already resolved to
itertools 0.14; only our workspace pin held a second 0.13 copy in the
graph. Bumping the pin collapses it (criterion keeps 0.13 for dev/bench
only). No API changes needed — we use no removed 0.13 items.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@wjones127
wjones127 force-pushed the chore/upgrade-itertools-rand branch from b25d8b2 to 343a97e Compare June 23, 2026 16:54
@github-actions github-actions Bot added the A-java Java bindings + JNI label Jun 23, 2026
@wjones127 wjones127 changed the title chore: upgrade itertools to 0.14 deps: upgrade itertools to 0.14 Jun 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

ACTION NEEDED
Lance follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

For details on the error please inspect the "PR Title Check" action.

@wjones127 wjones127 changed the title deps: upgrade itertools to 0.14 chore: upgrade itertools to 0.14 Jun 23, 2026
@codecov

codecov Bot commented Jun 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@wjones127
wjones127 marked this pull request as ready for review June 23, 2026 17:38
@wjones127
wjones127 requested review from Xuanwo and hamersaw June 23, 2026 17:38
@wjones127
wjones127 merged commit fa926e2 into lance-format:main Jun 23, 2026
36 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-deps Dependency updates A-java Java bindings + JNI A-python Python bindings chore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants