Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/950#discussion_r140134229
--- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicClient.java
---
@@ -120,6 +125,25 @@ protected void initChannel(SocketChannel ch) throws
Exception {
// }
}
+ // Adds a SSL handler if enabled. Required only for client and server
communications, so
+ // a real implementation is only available for UserClient
+ protected void setupSSL(ChannelPipeline pipe,
ConnectionMultiListener.SSLHandshakeListener sslHandshakeListener) {
+ // Do nothing
--- End diff --
we can throw UnSupportedException to catch any error in flows where this is
called in context of Data/control server.
---
