You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by jtstorck <gi...@git.apache.org> on 2018/01/02 23:41:42 UTC

[GitHub] nifi pull request #2360: NIFI-4323 Wrapped Get/ListHDFS hadoop operations in...

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

    https://github.com/apache/nifi/pull/2360#discussion_r159340068
  
    --- Diff: nifi-nar-bundles/nifi-extension-utils/nifi-hadoop-utils/src/main/java/org/apache/nifi/hadoop/SecurityUtil.java ---
    @@ -51,7 +50,8 @@ public static synchronized UserGroupInformation loginKerberos(final Configuratio
             Validate.notNull(keyTab);
     
             UserGroupInformation.setConfiguration(config);
    -        return UserGroupInformation.loginUserFromKeytabAndReturnUGI(principal.trim(), keyTab.trim());
    +        UserGroupInformation.loginUserFromKeytab(principal.trim(), keyTab.trim());
    --- End diff --
    
    I've added javadoc to several methods that are affected by this change.  Will update the PR with these changes.


---