lidavidm commented on issue #37937: URL: https://github.com/apache/arrow/issues/37937#issuecomment-1766703436
Yes: - Right now, we do a cast that is definitely UB to try to hook into the gRPC serialization machinery - gRPC offers a "generic" stub that gives us the raw byte buffers directly without that hack, but it is only available for the async API - The gRPC serialization machinery is known to be broken (e.g. if you raise an error gRPC will just abort instead) so being able to bypass it entirely is a benefit So when we implement the async version of DoGet etc we should just try to use the generic stub and avoid all this in the first place (requires some refactoring, though) -- 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]
