[ 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