----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74259/#review226343 -----------------------------------------------------------
kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStoreProvider.java Lines 173 (patched) <https://reviews.apache.org/r/74259/#comment314588> RangerAWSKMSProvider class is missing in last uploaded patch. Can you please update the patch with RangerAWSKMSProvider.java - Dhaval Shah On March 21, 2024, 9:21 a.m., Kirby Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74259/ > ----------------------------------------------------------- > > (Updated March 21, 2024, 9:21 a.m.) > > > Review request for ranger, Dhaval Shah, Dineshkumar Yadav, Gautam Borad, > Jayendra Parab, Kishor Gollapalliwar, Abhay Kulkarni, Mateen Mansoori, Mehul > Parikh, Pradeep Agrawal, VaradreawiZTV VaradreawiZTV, Vishal Suvagia, and > Velmurugan Periasamy. > > > Bugs: ranger-3998 > https://issues.apache.org/jira/browse/ranger-3998 > > > Repository: ranger > > > Description > ------- > > AWS KMS is widely used by many customers. > Therefore, RangerKMS should support hosting MasterKey to AWS KMS. > > > Diffs > ----- > > distro/src/main/assembly/kms.xml 77f61b917 > kms/config/kms-webapp/dbks-site.xml 258d331d8 > kms/pom.xml 2739bb81a > kms/scripts/install.properties 5f5df9b1f > kms/scripts/setup.sh 0cce478fd > kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStoreProvider.java > 429d1ce45 > > kms/src/main/java/org/apache/hadoop/crypto/key/RangerTencentKMSProvider.java > 1f0125967 > > > Diff: https://reviews.apache.org/r/74259/diff/7/ > > > Testing > ------- > > Tested under macos-13 and centos-8 > > > File Attachments > ---------------- > > 0001-RANGER-3998-KMS-integration-with-AWS-KMS.patch > > https://reviews.apache.org/media/uploaded/files/2023/04/07/6499bf0e-af49-4eae-8636-f183f0fab3d7__0001-RANGER-3998-KMS-integration-with-AWS-KMS.patch > > > Thanks, > > Kirby Zhou > >