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

Nandakumar commented on HDFS-12147:
-----------------------------------

Thanks [~vagarychen] for the review.

bq. Maybe we should change the doc a little bit?
I will change the doc accordingly.
bq. void checkBucketAccess(...) throws AccessControlException
According to {{StorageHandler}} definition  {{checkBucketAccess}} originally 
had void as return type, I wanted {{checkBucketAccess}} to be consistent with 
{{checkVolumeAccess}} that is the reason for having boolean as return type.
I'm ok with {{void checkBucketAccess(...) throws AccessControlException}}, will 
file a jira to change {{checkVolumeAccess}}'s behavior.

> Ozone: KSM: Add checkBucketAccess
> ---------------------------------
>
>                 Key: HDFS-12147
>                 URL: https://issues.apache.org/jira/browse/HDFS-12147
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>            Reporter: Nandakumar
>            Assignee: Nandakumar
>         Attachments: HDFS-12147-HDFS-7240.000.patch, 
> HDFS-12147-HDFS-7240.001.patch
>
>
> Checks if the caller has access to a given bucket.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
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