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

Rakesh R commented on HDFS-7529:
--------------------------------

[~wheat9], I'd like to take this ahead to the completion. As per the offline 
discussion, am attaching a patch. I hope I've addressed the above comments. 
Please take a look at the patch when you get a chance. Thanks!

Following are the major changes:
- I feel {{provider.getMetadata()}} is not required to do twice as the 
'provider' has {{provider.warmUpEncryptedKeys(keyName);}} warming up logic. 
Does this sound good to you ?
- modified few codes under the lock section.
- rebased in latest trunk code

> Consolidate encryption zone related implementation into a single class
> ----------------------------------------------------------------------
>
>                 Key: HDFS-7529
>                 URL: https://issues.apache.org/jira/browse/HDFS-7529
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Haohui Mai
>            Assignee: Haohui Mai
>              Labels: BB2015-05-TBR
>         Attachments: HDFS-7529-002.patch, HDFS-7529.000.patch, 
> HDFS-7529.001.patch
>
>
> This jira proposes to consolidate encryption zone related implementation to a 
> single class.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to