-
Notifications
You must be signed in to change notification settings - Fork 901
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
Fix debug warnings/errors in from_arrow_device_test.cpp #15596
Fix debug warnings/errors in from_arrow_device_test.cpp #15596
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like C++ INTEROP_TEST
is failing.
Waiting for the C++ tests to pass to approve.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for investigating this in #15597 and fixing the tests. Looks good to me!
/merge |
Description
Fixes debug build errors introduced by #15458
These warnings show up in a debug build where warnings become errors.
Some of the errors:
Closes #15597
Checklist