Github user sohami commented on a diff in the pull request:
--- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicClient.java
@@ -69,6 +74,11 @@
private final IdlePingHandler pingHandler;
sslHandshakeListener = null;
+ // Authentication related parameters
+ protected volatile List<String> serverAuthMechanisms = null;
--- End diff --
On second thought volatile doesn't seem necessary here since it will only
be accessed by Netty's thread which is also fixed for a connection.
Made fields private.