[GitHub] [hadoop] steveloughran commented on a change in pull request #823: HADOOP-16315. ABFS: transform full UPN for named user in AclStatus

[GitBox <gi...@apache.org>]

steveloughran commented on a change in pull request #823: HADOOP-16315. ABFS: transform full UPN for named user in AclStatus
URL: https://github.com/apache/hadoop/pull/823#discussion_r291150581
 
 

 ##########
 File path: hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/oauth2/IdentityTransformer.java
 ##########
 @@ -218,7 +217,62 @@ public String transformUserOrGroupForSetRequest(String userOrGroup) {
       // Replace the original AclEntry
       aclEntries.set(i, aclEntryBuilder.build());
     }
-    return aclEntries;
+  }
+
+  /**
+   * Perform Identity transformation when calling GetAclStatus()
+   * If the AclEntry type is a user or group, and its name is one of the following:
+   * 1. $superuser:
 
 Review comment:
   surround with `pre` tags so that in javadocs this comes out formatted.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org