xiaoyuyao commented on a change in pull request #2008:
URL: https://github.com/apache/ozone/pull/2008#discussion_r596270422
##########
File path:
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/security/acl/RequestContext.java
##########
@@ -43,20 +43,23 @@
*/
private final boolean recursiveAccessCheck;
+ private final OzonePrefixPath ozonePrefixPath;
+
@SuppressWarnings("parameternumber")
public RequestContext(String host, InetAddress ip,
UserGroupInformation clientUgi, String serviceId,
ACLIdentityType aclType, ACLType aclRights,
String ownerName) {
this(host, ip, clientUgi, serviceId, aclType, aclRights, ownerName,
- false);
+ false, null);
}
@SuppressWarnings("parameternumber")
public RequestContext(String host, InetAddress ip,
UserGroupInformation clientUgi, String serviceId,
ACLIdentityType aclType, ACLType aclRights,
- String ownerName, boolean recursiveAccessCheck) {
+ String ownerName, boolean recursiveAccessCheck,
+ OzonePrefixPath ozonePrefixPath) {
Review comment:
Should we put the OzonePrefixPath in the OzoneObject with a flag here in
the RequestContext indicating the check is for prefixPath?
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
