[GitHub] flink pull request #4636: [FLINK-2268] Allow Flink binary release without Ha...

[aljoscha]

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

    https://github.com/apache/flink/pull/4636#discussion_r137212834
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/security/KerberosUtils.java
 ---
    @@ -52,6 +51,13 @@
     
        private static final AppConfigurationEntry userKerberosAce;
     
    +   /* Return the Kerberos login module name */
    +   public static String getKrb5LoginModuleName() {
    --- End diff --
    
    Yes, this had a dependency on `KerberosUtil` form Hadoop just for this 
method. Now we can have Kerberos independent of Hadoop.


---