-
Notifications
You must be signed in to change notification settings - Fork 770
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
Java Integration Tests Failing In CI #5815
Comments
@lidavidm perhaps you might know what is causing this, or know someone who might (I'm just going off the git history of the Java arrow impl) 😅 |
The upstream pipeline seems fine... |
@vibhatha seems like the recent changes did affect IPC? But in a way that only affects Rust? |
Yes, I looked into the code, arrow-rs/arrow-ipc/src/reader.rs Line 590 in 7fe01bb
variadic_count and it is only failing one branch of the code (else branch).
In Java, we added the missing spec attribute |
Why would variadicBufferCounts be populated for payloads that don't contain variadic data?
This possibly shouldn't be an assertion failure in Rust but a regular error, but the data Java is producing is malformed is it not? |
Can't they just all be 0? |
The specification states that they should only contain entries for the ones that contain variadic buffers - https://github.com/apache/arrow-rs/blob/master/format/Message.fbs#L103 |
Hmm, Java is probably wrong then |
I've filled apache/arrow#41902 |
@tustvold we have marged a fix for the filed issue. Could you please check the CIs? |
All green now, thank you |
Java Integration tests are failing with some strange error about not being able to find flight
https://github.com/apache/arrow-rs/actions/runs/9284625938/job/25547387572
The text was updated successfully, but these errors were encountered: