ctubbsii commented on a change in pull request #1030: Kerberos support #1025
URL: https://github.com/apache/fluo/pull/1030#discussion_r177888844
 
 

 ##########
 File path: 
modules/core/src/main/java/org/apache/fluo/core/client/FluoAdminImpl.java
 ##########
 @@ -76,6 +77,28 @@
 
   private final String appRootDir;
 
+  /**
+   * Kerberos autentication method.
+   * 
+   * @param realm Realm to be used in authentication.
+   * @param keytab Keytab path.
+   * @since 1.3.0
+   */
+  public void loginWithKerberos(final String realm, final String keytab) {
 
 Review comment:
   Oh, interesting. I'm surprised other components worked from the environment, 
but this did not. I wonder why. Did you try fluo-yarn? Also, do you know if 
this change allows it to work with Accumulo with Kerberos tokens? Or did you 
only try Accumulo using regular PasswordTokens?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to