Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DRAFT] Update Calcite to 1.38 #2961

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

cgivre
Copy link
Contributor

@cgivre cgivre commented Nov 15, 2024

DRILL-XXXX: Update Calcite to 1.38

Description

This PR updates Apache Calcite to the latest branch.

Documentation

No user facing changes.

Testing

Running existing unit tests.

@cgivre cgivre added dependencies calcite-update Changes required for updating the Calcite version labels Nov 15, 2024
@cgivre cgivre self-assigned this Nov 15, 2024
@cgivre
Copy link
Contributor Author

cgivre commented Nov 15, 2024

My goal here is to work through the recent versions until we get to the latest.

I thought I'd post progress here. This branch will successfully build using version 1.35. However, Calcite made some changes to VARDECIMAL handling so several tests that were introduced in #2597 are now failing.

Any assistance would be greatly appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
calcite-update Changes required for updating the Calcite version dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant