umamaheswararao commented on code in PR #4474:
URL: https://github.com/apache/hadoop/pull/4474#discussion_r907908052


##########
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/ShellBasedUnixGroupsMapping.java:
##########
@@ -215,7 +215,13 @@ private Set<String> getUnixGroups(String user) throws 
IOException {
           groups = resolvePartialGroupNames(user, e.getMessage(),
               executor.getOutput());
         } catch (PartialGroupNameException pge) {
-          LOG.warn("unable to return groups for user {}", user, pge);
+          if(LOG.isDebugEnabled()) {

Review Comment:
   @swamirishi could you please fix the @smengcl comments above? you need to 
format line number 218



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to