You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Madhan Neethiraj <ma...@apache.org> on 2024/07/09 07:47:02 UTC

Review Request 75078: RANGER-4845: removed guava dependency

-----------------------------------------------------------
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


Re: Review Request 75078: RANGER-4845: removed guava dependency

Posted by zhongwei wang <gr...@163.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/75078/#review226643
-----------------------------------------------------------


Ship it!




Ship It!

- zhongwei wang


On 七月 9, 2024, 7:46 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/75078/
> -----------------------------------------------------------
> 
> (Updated 七月 9, 2024, 7:46 a.m.)
> 
> 
> 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
> 
>


Re: Review Request 75078: RANGER-4845: removed guava dependency

Posted by bhavik patel <bh...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/75078/#review226642
-----------------------------------------------------------


Ship it!




Ship It!

- bhavik patel


On July 9, 2024, 7:46 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/75078/
> -----------------------------------------------------------
> 
> (Updated July 9, 2024, 7:46 a.m.)
> 
> 
> 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
> 
>