-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HDDS-1975. Implement default acls for bucket/volume/key for OM HA code. #1315
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
2f3500c
to
20c802a
Compare
/retest |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
/retest |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
7a2b8e0
to
66be275
Compare
Updated and rebased the patch as now HDDS-1927 got checked in. |
66be275
to
17ba03b
Compare
/retest |
985653a
to
613731b
Compare
💔 -1 overall
This message was automatically generated. |
...e-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/OMBucketCreateRequest.java
Outdated
Show resolved
Hide resolved
...p-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OMKeyRequest.java
Show resolved
Hide resolved
...p-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OMKeyRequest.java
Show resolved
Hide resolved
...anager/src/main/java/org/apache/hadoop/ozone/om/request/s3/bucket/S3BucketDeleteRequest.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM overall. A few comments added inline.
Thank You @xiaoyuyao for the review. |
💔 -1 overall
This message was automatically generated. |
+1, Thanks @bharatviswa504 for the update. |
Thank You @xiaoyuyao for the review. |
No description provided.