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

Larry McCay commented on HADOOP-10646:
--------------------------------------

Hi [~tucu00] - After looking at it more closely, I'm not sure that the methods 
should be in the same utils class. BuildVersion is pretty much key  provider 
specific where unnestUri is used across both Key and CredentialProvider APIs. I 
added unnestUri to a new org.apache.hadoop.security.ProviderUtils class - since 
it is more general security provider functionality. Thanks.

> KeyProvider buildVersionName method should be moved to a utils class
> --------------------------------------------------------------------
>
>                 Key: HADOOP-10646
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10646
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: security
>    Affects Versions: 3.0.0
>            Reporter: Alejandro Abdelnur
>            Assignee: Alejandro Abdelnur
>             Fix For: 3.0.0
>
>
> The buildVersionName() method should not be part of the KeyProvider public 
> API because keyversions could be opaque (not built based on the keyname and 
> key generation counter).
> KeyProvider implementations may choose to use buildVersionName() for reasons 
> such as described in HADOOP-10611.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to