-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
GH-41717: [Java][Vector] fix issue with ByteBuffer rewind in MessageSerializer #41718
Conversation
|
@github-actions crossbow submit -g java |
|
@kou this error is not familiar. Could be a temporary issue? |
@github-actions crossbow submit -g java |
|
cc @raulcd @assignUser any idea why the crossbow bot command is failing? |
Uhh, looks like a new version of pygit? I'll have to check |
There was a release of pygit2 (1.15.0) yesterday so that checks. @vibhatha can you open an issue? |
The error on the job can be seen here: https://github.com/apache/arrow/actions/runs/9150811236/job/25156057745#step:5:30 |
What happened in 16.1.0, I got another similar error(in a different place)
|
@github-actions crossbow submit -g java |
Revision: f47bbcf Submitted crossbow builds: ursacomputing/crossbow @ actions-3f91e4c81b |
Those failures are mostly expected. #41628 should help if I can get an actual review. |
@pan3793 still facing this issue? |
After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit 1cd2872. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 29 possible false positives for unstable benchmarks that are known to sometimes produce them. |
…ssageSerializer (apache#41718) ### Rationale for this change ### What changes are included in this PR? apache#41717 describes issue and change ### Are these changes tested? CI build ### Are there any user-facing changes? * GitHub Issue: apache#41717 Authored-by: PJ Fanning <pjfanning@users.noreply.github.com> Signed-off-by: David Li <li.davidm96@gmail.com>
Rationale for this change
What changes are included in this PR?
#41717 describes issue and change
Are these changes tested?
CI build
Are there any user-facing changes?