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

Pakapoj Tulsuk commented on HDDS-5307:
--------------------------------------

Hi [~sombat.cha] Ill change 

{{checkAclRights}} to {{checkAclRight}} in {{KeyManagerImpl}} b/c it's easier

> Two functions performing the same logic
> ---------------------------------------
>
>                 Key: HDDS-5307
>                 URL: https://issues.apache.org/jira/browse/HDDS-5307
>             Project: Apache Ozone
>          Issue Type: Improvement
>          Components: OM
>    Affects Versions: 1.2.0
>            Reporter: Sombat Charnchaleo
>            Priority: Trivial
>              Labels: patch
>
> I found there are two functions performing the same purpose and the same 
> logic. Would like to have a review if we could reduce to only one function.
> *java source code:*
> java/org/apache/hadoop/ozone/om/helpers/OzoneAclUtil.java
> *class method:*
> checkAclRight() and checkAclRights()



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to