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

Fixup: Update duckdb to get Latest Relation Definition #122

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

anshuldata
Copy link
Contributor

@anshuldata anshuldata commented Nov 13, 2024

  • Update Workflow definition to use fix commit for duckdb submodule
  • Fix extension code required because of update of duckdb
  • Exclused "windows_amd64_rtools" from architecture to built. When duckdb version set to "main" Workflow didn't run windows arch in metrics generated here. If set to specific version, workflow started building windows architecture so to skip it explicitly added added "windows_amd64_rtools" (link) too from arch
  • Fix Workflow to use specific CI commit, so that it doesn't break with any change in future

* Update Workflow definition to use fix commit for duckdb submodule
* Fix extension code required because of update of duckdb
@anshuldata anshuldata changed the title Update duckdb to get Latest Relation Update duckdb to get Latest Relation Definition Nov 13, 2024
@anshuldata anshuldata changed the title Update duckdb to get Latest Relation Definition Fixup: Update duckdb to get Latest Relation Definition Nov 13, 2024
@jacques-n jacques-n merged commit 6ae6557 into substrait-io:main Nov 13, 2024
9 checks passed
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.

3 participants