-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74221/
-----------------------------------------------------------
(Updated Nov. 30, 2022, 8:49 p.m.)
Review request for ranger, Ankita Sinha, Don Bosco Durai, Kishor Gollapalliwar,
Abhay Kulkarni, Mehul Parikh, Monika Kachhadiya, Pradeep Agrawal, Ramesh Mani,
Siddhesh Phatak, Sailaja Polavarapu, Subhrat Chaudhary, and Velmurugan
Periasamy.
Changes
-------
fixed leaking threads in AzureKeyVaultClientAuthenticator
Bugs: RANGER-3989
https://issues.apache.org/jira/browse/RANGER-3989
Repository: ranger
Description
-------
- addressed concurrency issues in dealing with simultaneous requests
- JPA object loading improvements, to avoid cache overheads
- code readability improvements
Diffs (updated)
-----
kms/config/kms-webapp/kms-logback.xml 16e82a656
kms/src/main/java/org/apache/hadoop/crypto/key/AzureKeyVaultClientAuthenticator.java
13897ac3e
kms/src/main/java/org/apache/hadoop/crypto/key/RangerKMSDB.java 812e7dfc4
kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java 8dc129069
kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStoreProvider.java
cb5739f61
kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java c37e98ee5
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMS.java c83382d64
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KeyAuthorizationKeyProvider.java
d9f1b5b8e
kms/src/main/java/org/apache/ranger/kms/biz/RangerKMSStartUp.java aae722b39
kms/src/main/java/org/apache/ranger/kms/dao/BaseDao.java f97154cb9
kms/src/main/java/org/apache/ranger/kms/dao/DaoManager.java c28e78535
kms/src/main/java/org/apache/ranger/kms/dao/RangerKMSDao.java cb64310c2
kms/src/main/java/org/apache/ranger/kms/dao/RangerMasterKeyDao.java 22edc9a13
kms/src/main/resources/META-INF/kms_jpa_named_queries.xml 94d5fa67c
Diff: https://reviews.apache.org/r/74221/diff/2/
Changes: https://reviews.apache.org/r/74221/diff/1-2/
Testing
-------
Called KMS APIs simultaneously from multiple threads and verified that failures
seen with concurrent requests prior to this patch are resolved with this patch.
Thanks,
Madhan Neethiraj
