[ 
https://issues.apache.org/jira/browse/HDFS-11778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16022320#comment-16022320
 ] 

Nandakumar commented on HDFS-11778:
-----------------------------------

Thanks for the review [~xyao].
bq. I think we can just keep using the BucketInfo as the parameter for 
CreateBucketRequest. The add/remove ACL is mainly for setBucket. BucketInfo is 
good enough for CreateBucket/GetBuckert.
I was also thinking of removing KsmBucketArgs, and use KsmBucketInfo for 
CreateBucketRequest. Initially KsmBucketArgs was added to make it consistent 
with {{web.handlers.BucketArgs}}
bq. bucketInfo has an aclList which will usually not needed to coexist with 
addAcls and removeAcls. Any reason for changing the current KsmBucketArgs?
Yes, true. Since KsmBucketArgs  was used for createBucket, bucketInfo was 
added; that can be removed since we are going to directly use KsmBucketInfo for 
create call.

I will rework and upload a new patch soon.



> Ozone: KSM: add getBucketInfo
> -----------------------------
>
>                 Key: HDFS-11778
>                 URL: https://issues.apache.org/jira/browse/HDFS-11778
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>    Affects Versions: HDFS-7240
>            Reporter: Anu Engineer
>            Assignee: Nandakumar
>         Attachments: HDFS-11778-HDFS-7240.000.patch, 
> HDFS-11778-HDFS-7240.001.patch
>
>
> Returns the bucket information if the bucket exists.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to