carols10cents commented on pull request #8962: URL: https://github.com/apache/arrow/pull/8962#issuecomment-748281158
I made the same sort of fix in the spot you pointed out, to each of the 3 byte fields in `FlightData`. What do you think? > Note I took a look at the Java side of this and the fix may not be so simple, since some code paths in Java expect there to be 0 buffers, and providing a default buffer breaks them; C++ may have the same issue. Yuck. Should I open a new Jira ticket to track the Java side or should this ticket cover the same problem in the Java and C++ implementations? > (Those paths are arguably also wrong, since Protobuf implementations can write the empty field...) Hm, isn't the C++ client currently writing the empty field? Or is it writing the field to `null`, rather than an empty list of bytes? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
