tustvold commented on issue #5731: URL: https://github.com/apache/arrow-rs/issues/5731#issuecomment-2123047194
My 2 cents is that type-erased payloads should always be `Any`. Protobuf is designed to ignore message content that it doesn't expect to allow for API evolution, and a consequence of this is decoding a message as an incorrect type does not necessarily lead to an error. By ensuring such payloads are consistently wrapped in `Any` this form of silent failure can be avoided, payloads are at least somewhat self-describing, and we preserve some form of type safety. Now I agree it is unfortunate that FlightSQL was designed in this way, gRPC provides a type-safe DSL and the layering of FlightSQL on top of Flight sacrificed many of these benefits, but the least we can do is ensure that users get a sensible runtime error.... -- 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]
