Skip to content

chore(deps): bump substrait packages to 0.103.1 - #299

Merged
nielspardon merged 1 commit into
substrait-io:mainfrom
nielspardon:chore/bump-substrait-0.103.1
Oct 13, 2026
Merged

nielspardon merged 1 commit into
substrait-io:mainfrom
nielspardon:chore/bump-substrait-0.103.1

Conversation

@nielspardon

Copy link
Copy Markdown
Member

Closes #273.

Bumps substrait-protobuf, substrait-extensions, and substrait-antlr to spec v0.103.1 (release notes).

The only change is that quantile now declares its distribution argument as any1 and returns LIST?<any1> instead of an unbound LIST?<any>. The registry could not derive a return type from the unbound any, so quantile lookups previously failed to resolve; they now resolve to a list of the distribution's type. The compact signature is unchanged. A regression test covers the new lookup.

🤖 Generated with AI

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: substrait-io/substrait-python/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5aeaf5cd-02eb-4f26-80ea-e72f6b61a06b

📥 Commits

Reviewing files that changed from the base of the PR and between b812394 and 9169bd0.


⛔ Files ignored due to path filters (2)
  • pixi.lock is excluded by !**/*.lock
  • uv.lock is excluded by !**/*.lock

📒 Files selected for processing (2)
  • pyproject.toml
  • tests/extension_registry/test_registry_lookup.py

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Summary

Summary by CodeRabbit

  • Bug Fixes
    • Fixed arithmetic quantile lookup so it resolves with the expected nullable list result type.
  • Chores
    • Updated Substrait dependencies to version 0.103.1.

Walkthrough

The project dependency pins move from Substrait 0.103.0 to 0.103.1. A registry lookup test checks the return type of arithmetic quantile.

Changes

Substrait 0.103.1 Update

Layer / File(s) Summary
Dependency pins and quantile lookup test
pyproject.toml, tests/extension_registry/test_registry_lookup.py
The project, optional, and development dependency pins use version 0.103.1. A regression test checks that arithmetic quantile resolves to a nullable list of non-nullable fp64.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other


Merge Risk: ⚪ Minimal · up to 9169b

The dependency update and quantile type regression check show no actionable merge-blocking risk in the reviewed changes.

Pre-merge checks | Passed 4 | Failed 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files. (1 skipped: 1 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check Passed The title is a valid Conventional Commit and clearly identifies the dependency update to Substrait packages version 0.103.1.
Description check Passed The description provides the issue reference, rationale, affected dependencies, functional impact of the quantile change, and regression-test coverage. It satisfies the available template requiremen…
Linked Issues check Passed Issue #273 requires an update to Substrait v0.103.1. pyproject.toml updates substrait-protobuf, substrait-extensions, and substrait-antlr from 0.103.0 to 0.103.1. The regression test in `tests…
Out of Scope Changes check Passed The changes are limited to the three dependency updates required by issue #273 and a regression test for the affected quantile lookup. The test directly supports the dependency update and does not a…

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files. (1 skipped: 1 unsupported.)


  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@nielspardon

Copy link
Copy Markdown
Member Author

@coderabbitai resume

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
✅ Action performed

Reviews resumed and review finished.

@nielspardon
nielspardon merged commit 26322a2 into substrait-io:main Oct 13, 2026
22 checks passed
@nielspardon
nielspardon deleted the chore/bump-substrait-0.103.1 branch October 13, 2026 08:02
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.

Update to Substrait v0.103.1

2 participants