You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Ankita Sinha <an...@freestoneinfotech.com> on 2016/06/07 09:39:00 UTC

Review Request 48331: Ranger KMS changes to cater to hadoop kms commits

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48331/
-----------------------------------------------------------

Review request for ranger, Don Bosco Durai, Gautam Borad, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.


Bugs: RANGER-1015
    https://issues.apache.org/jira/browse/RANGER-1015


Repository: ranger


Description
-------

**Problem Statement**

As below given changes are committed in Hadoop KMS, need to cater to these changes in Ranger KMS as well.
1. [ jdk8 ] java doc issue for "KeyAuthorizationKeyProvider.java"
2. ThreadLocal initialization in several classes is not thread safe
3. Improving Ranger KMS log for server failure.


Diffs
-----

  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSExceptionsProvider.java 5cb0885 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMDCFilter.java 6352fbc 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KeyAuthorizationKeyProvider.java fe2f175 

Diff: https://reviews.apache.org/r/48331/diff/


Testing
-------

1. Tested Ranger KMS with Key Operation from Ranger UI, Hadoop Command and CURL Command
2. Tested Multiple Ranger KMS working


Thanks,

Ankita Sinha


Re: Review Request 48331: Ranger KMS changes to cater to hadoop kms commits

Posted by Velmurugan Periasamy <vp...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48331/#review136462
-----------------------------------------------------------


Ship it!




Ship It!

- Velmurugan Periasamy


On June 7, 2016, 9:38 a.m., Ankita Sinha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48331/
> -----------------------------------------------------------
> 
> (Updated June 7, 2016, 9:38 a.m.)
> 
> 
> Review request for ranger, Don Bosco Durai, Gautam Borad, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-1015
>     https://issues.apache.org/jira/browse/RANGER-1015
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> **Problem Statement**
> 
> As below given changes are committed in Hadoop KMS, need to cater to these changes in Ranger KMS as well.
> 1. [ jdk8 ] java doc issue for "KeyAuthorizationKeyProvider.java"
> 2. ThreadLocal initialization in several classes is not thread safe
> 3. Improving Ranger KMS log for server failure.
> 
> 
> Diffs
> -----
> 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSExceptionsProvider.java 5cb0885 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMDCFilter.java 6352fbc 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KeyAuthorizationKeyProvider.java fe2f175 
> 
> Diff: https://reviews.apache.org/r/48331/diff/
> 
> 
> Testing
> -------
> 
> 1. Tested Ranger KMS with Key Operation from Ranger UI, Hadoop Command and CURL Command
> 2. Tested Multiple Ranger KMS working
> 
> 
> Thanks,
> 
> Ankita Sinha
> 
>