adamreeve commented on PR #41264: URL: https://github.com/apache/arrow/pull/41264#issuecomment-2065465259
Hi, sorry for breaking this! I didn't think to also run the integration tests. The Java tests are unrelated to the validity buffer validation but are throwing an error when writing zero length binary arrays to IPC format. When we create a zero-length binary or list array in C# we get an offsets buffer with length 1, but the arrays read from Java have zero-length offsets. I've pushed a fix for this a branch in my fork: https://github.com/adamreeve/arrow/commit/b2bfb7453d437fd52986a08a48ae2c1a2bc8708d. Do you want to include that in this PR or should I make a separate one for that? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
