Skip to content

Conversation

@jqnatividad
Copy link
Collaborator

related to #3018

@jqnatividad jqnatividad requested a review from Copilot September 30, 2025 11:51
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR bumps the Polars dependency to version 0.51.0 using the py-1.34.0-beta.4 tag at revision b973cac to stay current with the latest upstream changes.

  • Updates Polars dependency from py-1.33.1 to py-1.34.0-beta.4 at revision b973cac
  • Adapts code to handle breaking changes in Polars API, particularly around decimal types and optimization flags
  • Updates test expectations to match new numeric precision behavior

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Cargo.toml Updates Polars dependencies to py-1.34.0-beta.4 at revision b973cac
src/util.rs Fixes Decimal type constructor and formats user agent string
src/cmd/sqlp.rs Updates IpcCompression::ZSTD usage and removes deprecated COLLAPSE_JOINS flag
src/cmd/joinp.rs Removes deprecated COLLAPSE_JOINS optimization flag
src/cmd/schema.rs Updates documentation to mention explicit precision and scale for Decimal types
tests/test_sqlp.rs Updates test expectations to match new numeric precision behavior

@jqnatividad jqnatividad requested a review from Copilot September 30, 2025 12:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.

@jqnatividad jqnatividad merged commit d0fa8f0 into master Sep 30, 2025
16 checks passed
@jqnatividad jqnatividad deleted the polars-0.51.0-latest-upstream branch September 30, 2025 14:33
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