[GitHub] drill pull request #1028: DRILL-5943: Avoid the strong check introduced by D...

[laurentgo]

Github user laurentgo commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1028#discussion_r150118518
  
    --- Diff: contrib/native/client/src/clientlib/saslAuthenticatorImpl.hpp ---
    @@ -59,6 +59,12 @@ class SaslAuthenticatorImpl {
     
         const char *getErrorMessage(int errorCode);
     
    +    static const std::string KERBEROS_SIMPLE_NAME;
    +
    +    static const std::string KERBEROS_SASL_NAME;
    --- End diff --
    
    do we need to expose it? (it looks like we only look for the keys)


---