StephanEwen commented on issue #6626: [FLINK-5029] [QueryableState] SSL Support URL: https://github.com/apache/flink/pull/6626#issuecomment-418114906 Thanks for this big contribution! I have a few questions of the overall approach: - How does this work with repeated queries/lookups? Is each query a new connection, with a new SSL handshake? Or is this more like a "streamed RPC", like grpc? - Can we avoid moving the SSLUtils etc to `flink-core`? With possible support for OpenSSL bindings, this would be nice to keep this complexity out of the core project. Otherwise, we would need a `flink-runtime` dependency in the `queryable-state-client`, so reverse the dependencies. Would that be feasible?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
