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

Review request for ranger, Abhishek  Kumar, Bhavik Bavishi, Fateh Singh, Kishor 
Gollapalliwar, Abhay Kulkarni, Mehul Parikh, Mugdha Varadkar, Pradeep Agrawal, 
Ramesh Mani, Sailaja Polavarapu, Velmurugan Periasamy, and zhongwei wang.


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


Repository: ranger


Description
-------

- replaced use of following guava classes/pacakges:
  -- com.google.common.base.Strings =>  org.apache.commons.lang3.StringUtils
-- com.google.common.base         => 
org.apache.hadoop.thirdparty.com.google.common.base
-- com.google.common.collect      => 
org.apache.hadoop.thirdparty.com.google.common.collect
-- com.google.common.annotations  => 
org.apache.hadoop.thirdparty.com.google.common.annotations
-- com.google.common.util.concurrent.ThreadFactoryBuilder => 
org.apache.hadoop.thirdparty.com.google.common.util.concurrent.ThreadFactoryBuilder
- updated pom files to remove dependency on guava library
- updated distro to remove packaging of guava library


Diffs
-----

  agents-audit/pom.xml 06e81650f 
  
agents-audit/src/main/java/org/apache/ranger/audit/destination/ElasticSearchAuditDestination.java
 b81c30e51 
  
agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceDefHelper.java
 51a3aae6f 
  
agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceDefValidator.java
 9cbea7f61 
  
agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceValidator.java
 5521146b9 
  agents-common/src/main/java/org/apache/ranger/plugin/util/PasswordUtils.java 
f1f2b0920 
  
agents-common/src/main/java/org/apache/ranger/plugin/util/PerfDataRecorder.java 
a1df53fac 
  
agents-common/src/test/java/org/apache/ranger/plugin/conditionevaluator/RangerTimeOfDayMatcherTest.java
 ec64e33e4 
  
agents-common/src/test/java/org/apache/ranger/plugin/errors/TestValidationErrorCode.java
 2c69bcc6f 
  
agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestDirectedGraph.java
 3ec20d99a 
  
agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerPolicyValidator.java
 7e16082bc 
  
agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerServiceDefHelper.java
 b0c108530 
  
agents-common/src/test/java/org/apache/ranger/plugin/resourcematcher/RangerDefaultResourceMatcherTest.java
 8a297bde0 
  
agents-common/src/test/java/org/apache/ranger/plugin/resourcematcher/RangerPathResourceMatcherTest.java
 3727b30d4 
  
agents-common/src/test/java/org/apache/ranger/plugin/resourcematcher/RangerURLResourceMatcherTest.java
 cc7307615 
  
agents-common/src/test/java/org/apache/ranger/plugin/util/PasswordUtilsTest.java
 d0b180645 
  credentialbuilder/pom.xml 0f61148ee 
  distro/src/main/assembly/admin-web.xml 0c8afa9d0 
  distro/src/main/assembly/hbase-agent.xml b36652091 
  distro/src/main/assembly/hdfs-agent.xml 498b2ff2b 
  distro/src/main/assembly/hive-agent.xml 17757f90b 
  distro/src/main/assembly/kms.xml c9b863088 
  distro/src/main/assembly/knox-agent.xml e97a9c6c8 
  distro/src/main/assembly/plugin-atlas.xml ed0108475 
  distro/src/main/assembly/plugin-elasticsearch.xml a627f641b 
  distro/src/main/assembly/plugin-kafka.xml 782f0ab79 
  distro/src/main/assembly/plugin-kms.xml 928db2041 
  distro/src/main/assembly/plugin-kylin.xml 968197a4a 
  distro/src/main/assembly/plugin-ozone.xml f1b78699c 
  distro/src/main/assembly/plugin-presto.xml b5fced211 
  distro/src/main/assembly/plugin-solr.xml fb05fa2af 
  distro/src/main/assembly/plugin-sqoop.xml 37c1a6f8f 
  distro/src/main/assembly/plugin-trino.xml a2f4828a4 
  distro/src/main/assembly/plugin-yarn.xml 9f112dcbf 
  distro/src/main/assembly/ranger-tools.xml 518615c9b 
  distro/src/main/assembly/sample-client.xml b7c73ef8b 
  distro/src/main/assembly/storm-agent.xml 1beb06c96 
  distro/src/main/assembly/tagsync.xml 616f577f1 
  distro/src/main/assembly/usersync.xml 7ade67859 
  hbase-agent/pom.xml 5b4e4313e 
  
hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/AuthorizationSession.java
 2e8eeafea 
  
hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessor.java
 cc9a68f27 
  
hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationFilter.java
 6e3900a18 
  
hbase-agent/src/test/java/org/apache/ranger/authorization/hbase/RangerAuthorizationFilterTest.java
 e009347b8 
  
hdfs-agent/src/main/java/org/apache/ranger/authorization/hadoop/RangerHdfsAuthorizer.java
 e226da8a5 
  
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java
 4ba327337 
  kms/pom.xml bfac6424a 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java 7b5e8bf7b 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMS.java 4f5da6471 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSAudit.java 
281ff4554 
  kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSWebApp.java 
f0ed18309 
  
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KeyAuthorizationKeyProvider.java
 e63154942 
  
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/SimpleKMSAuditLogger.java
 50a92b507 
  
knox-agent/src/test/java/org/apache/ranger/services/knox/RangerAdminClientImpl.java
 4c51e592e 
  plugin-atlas/pom.xml ad950f70d 
  
plugin-elasticsearch/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAuthorizer.java
 a09319f63 
  
plugin-kms/src/main/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizer.java
 799eb322c 
  plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java 
14e9c3bd1 
  
plugin-kylin/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java
 bcfaaafef 
  
plugin-ozone/src/main/java/org/apache/ranger/authorization/ozone/authorizer/RangerOzoneAuthorizer.java
 53cc0201e 
  
plugin-presto/src/test/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControlTest.java
 ed8c837d5 
  
plugin-solr/src/main/java/org/apache/ranger/authorization/solr/authorizer/FieldToAttributeMapping.java
 ba5f7d478 
  
plugin-solr/src/main/java/org/apache/ranger/authorization/solr/authorizer/RangerSolrAuthorizer.java
 e9b053381 
  
plugin-solr/src/main/java/org/apache/ranger/authorization/solr/authorizer/SubsetQueryPlugin.java
 1ea4d2843 
  
plugin-sqoop/src/main/java/org/apache/ranger/authorization/sqoop/authorizer/RangerSqoopAuthorizer.java
 65e08e41b 
  plugin-trino/pom.xml e4d161417 
  
plugin-yarn/src/main/java/org/apache/ranger/authorization/yarn/authorizer/RangerYarnAuthorizer.java
 de3493eb6 
  pom.xml 68fc34a72 
  ranger-common-ha/src/main/java/org/apache/ranger/ha/CuratorFactory.java 
758516a0e 
  ranger-common-ha/src/main/java/org/apache/ranger/ha/ServiceState.java 
6ae4a33f7 
  
ranger-common-ha/src/main/java/org/apache/ranger/ha/ZookeeperSecurityProperties.java
 01a034fab 
  ranger-examples/distro/src/main/assembly/plugin-sampleapp.xml 16f60bc1d 
  ranger-examples/distro/src/main/assembly/sample-client.xml 681df2f6a 
  ranger-examples/plugin-sampleapp/pom.xml d9823cd7b 
  ranger-examples/sample-client/pom.xml e87844acd 
  ranger-presto-plugin-shim/pom.xml 6cd835b7f 
  
ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControlFactory.java
 e446ada03 
  
ranger-tools/src/main/java/org/apache/ranger/policyengine/perftest/v2/RangerPolicyFactory.java
 cef7bd959 
  
ranger-tools/src/test/java/org/apache/ranger/policyengine/RangerPolicyEnginePerformanceTest.java
 187d20227 
  security-admin/pom.xml 5ca5028d9 
  security-admin/src/main/java/org/apache/ranger/biz/KmsKeyMgr.java 3adb7794a 
  security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java 
a8234325b 
  security-admin/src/main/java/org/apache/ranger/common/TimedExecutor.java 
b5c47d3e9 
  security-admin/src/main/java/org/apache/ranger/rest/SecurityZoneREST.java 
8f87d26fa 
  
security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKrbFilter.java
 29c1bcdef 
  
security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java
 694976d68 
  
storm-agent/src/main/java/org/apache/ranger/authorization/storm/StormRangerPlugin.java
 9cf352880 
  
storm-agent/src/main/java/org/apache/ranger/authorization/storm/authorizer/RangerStormAuthorizer.java
 fd42090b1 
  tagsync/pom.xml a1445fdd8 
  ugsync/pom.xml 54f793aa3 
  
ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java
 b1a6af183 
  
ugsync/src/main/java/org/apache/ranger/unixusersync/process/UnixUserGroupBuilder.java
 5525305b7 
  unixauthclient/pom.xml c7fc6f33e 
  unixauthservice/pom.xml c26738ff6 


Diff: https://reviews.apache.org/r/75078/diff/1/


Testing
-------

- built successfully with all unit tests
- using docker setup, verified that services Ranger/Usersync/Tagsync/KMS and 
plugins HDFS/Hive/HBase/Kafka/YARN/KMS are successfully deployed and working


Thanks,

Madhan Neethiraj

Reply via email to