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

Aaron T. Myers commented on HADOOP-10770:
-----------------------------------------

Patch looks pretty good to me, Tucu. Only one little nit - seems like you've 
got some excessive indentation in {{KMS#generateEncryptedKeys}}.

+1 otherwise.

> KMS add delegation token support
> --------------------------------
>
>                 Key: HADOOP-10770
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10770
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: security
>    Affects Versions: 3.0.0
>            Reporter: Alejandro Abdelnur
>            Assignee: Alejandro Abdelnur
>         Attachments: HADOOP-10770.patch, HADOOP-10770.patch, 
> HADOOP-10770.patch, HADOOP-10770.patch
>
>
> This is a follow up on HADOOP-10769 for KMS itself.



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

Reply via email to