lidavidm commented on pull request #7476: URL: https://github.com/apache/arrow/pull/7476#issuecomment-645984765
> @lidavidm Looks we need another level of abstraction above flight client `generic_options`. Otherwise, we have to expose grpc constants(backend implementation) to client code. See TODO in this patch. The intent of the option _is_ to expose gRPC-specific options and not have to abstract them away; there are lots of options and there's no point keeping up with all of them. This option is entirely backend-dependent and would be pointless on a non-gRPC implementation. > I think we could simply enable this option by default in the Flight client. Sharing the same TCP connection for all Flight clients doesn't like a good idea at all. I think that makes sense for this option. ---------------------------------------------------------------- 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]
