----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75324/ -----------------------------------------------------------
Review request for ranger. Bugs: RANGER-5053 https://issues.apache.org/jira/browse/RANGER-5053 Repository: ranger Description ------- - updated plugin-shim modules to comply with checkstyle requirements - replaced calls to activatePluginClassLoader() and deactivatePluginClassLoader() with PluginClassLoaderActivator, an AutoClosable object Diffs ----- ranger-atlas-plugin-shim/pom.xml 2674e5af4 ranger-atlas-plugin-shim/src/main/java/org/apache/ranger/authorization/atlas/authorizer/RangerAtlasAuthorizer.java ebfc018a5 ranger-elasticsearch-plugin-shim/pom.xml d2b6ae8c8 ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAccessControl.java f9996b02a ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAuthorizer.java 28b11acc0 ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/RangerElasticsearchPlugin.java 2af13f87f ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/action/filter/RangerSecurityActionFilter.java 945d6a17e ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/authc/user/UsernamePasswordToken.java c1df938db ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/rest/filter/RangerSecurityRestFilter.java 92215e5a7 ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/utils/RequestUtils.java 4f407cd9a ranger-hbase-plugin-shim/pom.xml b724b09d7 ranger-hbase-plugin-shim/src/main/java/com/xasecure/authorization/hbase/XaSecureAuthorizationCoprocessor.java ada572234 ranger-hbase-plugin-shim/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessor.java 35b680fd4 ranger-hbase-plugin-shim/src/main/test/org/apache/hadoop/hbase/security/access/RangerAccessControlListsTest.java 3e2171d14 ranger-hdfs-plugin-shim/pom.xml d465b9158 ranger-hdfs-plugin-shim/src/main/java/org/apache/ranger/authorization/hadoop/RangerHdfsAuthorizer.java b8adc3fd0 ranger-hive-plugin-shim/pom.xml c452b30e0 ranger-hive-plugin-shim/src/main/java/com/xasecure/authorization/hive/authorizer/XaSecureHiveAuthorizerFactory.java 2eade8ada ranger-hive-plugin-shim/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizerFactory.java f93700800 ranger-kafka-plugin-shim/pom.xml 517f794cd ranger-kafka-plugin-shim/src/main/java/org/apache/ranger/authorization/kafka/authorizer/RangerKafkaAuthorizer.java 51fed29cc ranger-kms-plugin-shim/pom.xml 9211102f0 ranger-kms-plugin-shim/src/main/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizer.java 7fa36ce79 ranger-knox-plugin-shim/pom.xml dff39d324 ranger-knox-plugin-shim/src/main/java/org/apache/ranger/authorization/knox/RangerPDPKnoxFilter.java a69134b82 ranger-knox-plugin-shim/src/main/java/org/apache/ranger/authorization/knox/deploy/RangerPDPKnoxDeploymentContributor.java a5a0865a7 ranger-kylin-plugin-shim/pom.xml 4680c974c ranger-kylin-plugin-shim/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java e91a4ce52 ranger-ozone-plugin-shim/pom.xml 587dda896 ranger-ozone-plugin-shim/src/main/java/org/apache/ranger/authorization/ozone/authorizer/RangerOzoneAuthorizer.java 59851e0ec ranger-plugin-classloader/src/main/java/org/apache/ranger/plugin/classloader/PluginClassLoaderActivator.java PRE-CREATION ranger-presto-plugin-shim/pom.xml d16865e32 ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/PrestoRangerPlugin.java fe707726d ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerConfig.java e0ab0f3ee ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java 40638701f ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControlFactory.java 3e8f53dde ranger-solr-plugin-shim/pom.xml 50a4caca2 ranger-solr-plugin-shim/src/main/java/org/apache/ranger/authorization/solr/authorizer/RangerSolrAuthorizer.java 3a10bc2af ranger-sqoop-plugin-shim/pom.xml 91f528d3f ranger-sqoop-plugin-shim/src/main/java/org/apache/ranger/authorization/sqoop/authorizer/RangerSqoopAuthorizer.java ea4471d2a ranger-storm-plugin-shim/pom.xml 60d8b058f ranger-storm-plugin-shim/src/main/java/com/xasecure/authorization/storm/authorizer/XaSecureStormAuthorizer.java 723dd79bc ranger-storm-plugin-shim/src/main/java/org/apache/ranger/authorization/storm/authorizer/RangerStormAuthorizer.java 50ff18aec ranger-yarn-plugin-shim/pom.xml f148ce7b1 ranger-yarn-plugin-shim/src/main/java/org/apache/ranger/authorization/yarn/authorizer/RangerYarnAuthorizer.java 2ae1ca753 Diff: https://reviews.apache.org/r/75324/diff/1/ Testing ------- - pull request #473: https://github.com/apache/ranger/pull/473/ Thanks, Madhan Neethiraj