-
Notifications
You must be signed in to change notification settings - Fork 86
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
Rust API test_arrow test fails due to Utf8Error on macos-11 GitHub-hosted runner #2005
Comments
I think I've figured out why it is failing -- the macos-11 runner uses Clang/LLVM 13.0.0 by default (https://github.com/actions/runner-images/blob/main/images/macos/macos-11-Readme.md), whereas we inexplicitly require 14. This is demonstrated by these two runs, one of which failed and one of which succeeded, which differ only by Clang/LLVM version: |
Never mind, it can also fail with Clang/LLVM 14: |
As some context for the error, it's being raised by this line in arrow-rs, where it's accessing the name field of the schema. That name field should either be what's set here as a hardcoded ascii string, or one of the property names as set here (the rust test failing has a string column called I think the problem is that the property names are owned by the We should also add a tests of |
It also happens on Windows as well: |
Yes, it can occur on all platforms, but I think the last time you updated the branch you're testing on was the commit before #2019 was merged. |
Oops, I had this page open and forgot to refresh it, so I wasn't aware that it has been fixed. Please ignore my previous comment. |
The test has failed 2 out of 2 times so far.
Logs and workflow files can be found here:
https://github.com/kuzudb/kuzu/actions/runs/6111754214/job/16587694174
https://github.com/kuzudb/kuzu/actions/runs/6113068279/job/16591808577
The text was updated successfully, but these errors were encountered: