Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1145#discussion_r172678483 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/control/ControlClient.java --- @@ -103,6 +96,23 @@ protected void handle(ControlConnection connection, int rpcType, ByteBuf pBody, connection.getCurrentHandler().handle(connection, rpcType, pBody, dBody, sender); } + @Override + protected void prepareSaslHandshake(final RpcConnectionHandler<ControlConnection> connectionListener) --- End diff -- The implementation seems to be common between Control and Data client, can it be unified here?
---