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

[Java] The offset buffer of empty BaseVariableWidthVector is not initiated before exporting #41609

Closed
viirya opened this issue May 9, 2024 · 3 comments

Comments

@viirya
Copy link
Member

viirya commented May 9, 2024

Describe the bug, including details regarding any error messages, version, and platform.

This is a follow up of #40038. In #40038, we fixed null offset buffer issue for BaseVariableWidthVector. For empty vector, instead of a empty offset buffer which turns to be a null buffer through C Data Interface, we export a non-empty offset which is supposed to contain zero value.

But the initialization code has a bug in the PR #40043, so the offset buffer is not initialized. Note that this is not a regression because the exported vector never works due to null offset buffer.

Component(s)

Java

@viirya
Copy link
Member Author

viirya commented May 13, 2024

Fixed this issue at arrow-rs. Closing this.

@viirya viirya closed this as completed May 13, 2024
@vibhatha
Copy link
Collaborator

@viirya did you get any chance to evaluate the update with Java. Interested in learning the outcome 🙂

@viirya
Copy link
Member Author

viirya commented May 14, 2024

@viirya did you get any chance to evaluate the update with Java. Interested in learning the outcome 🙂

Yes, I have verified the fix apache/arrow-rs#5756 can work with Java Arrow 16.0.0 by cherry-picking it in my forked branch of arrow-rs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants