----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74259/#review225323 -----------------------------------------------------------
kms/src/main/java/org/apache/hadoop/crypto/key/RangerAWSKMSProvider.java Lines 24 (patched) <https://reviews.apache.org/r/74259/#comment313994> Please consider importing individual classes instead of all. - Kishor Gollapalliwar On Jan. 17, 2023, 9:49 a.m., Kirby Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74259/ > ----------------------------------------------------------- > > (Updated Jan. 17, 2023, 9:49 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 4b4a2ac8e > kms/config/kms-webapp/dbks-site.xml 258d331d8 > kms/pom.xml e97b993d7 > kms/scripts/install.properties 0e5da3c75 > kms/scripts/setup.sh f723e09bb > kms/src/main/java/org/apache/hadoop/crypto/key/RangerAWSKMSProvider.java > PRE-CREATION > 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/5/ > > > Testing > ------- > > Tested under macos-13 and centos-8 > > > Thanks, > > Kirby Zhou > >
