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


Ship it!




Ship It!

- Abhay Kulkarni


On Nov. 13, 2019, 2:59 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71759/
> -----------------------------------------------------------
> 
> (Updated Nov. 13, 2019, 2:59 a.m.)
> 
> 
> Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Mehul Parikh, 
> Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and 
> Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2646
>     https://issues.apache.org/jira/browse/RANGER-2646
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> - created RangerPluginConfig, RangerAdminConfig and RangerAuditConfig - that 
> load specific configuration files
> - replaced references to RangerConfiguration.getInstance() with one of these 
> new class instances, depending on the context
> 
> 
> Diffs
> -----
> 
>   
> agents-common/src/main/java/org/apache/ranger/admin/client/AbstractRangerAdminClient.java
>  2bc755786 
>   
> agents-common/src/main/java/org/apache/ranger/admin/client/RangerAdminClient.java
>  9510888ce 
>   
> agents-common/src/main/java/org/apache/ranger/admin/client/RangerAdminRESTClient.java
>  86469fd5a 
>   
> agents-common/src/main/java/org/apache/ranger/authorization/hadoop/config/RangerAdminConfig.java
>  PRE-CREATION 
>   
> agents-common/src/main/java/org/apache/ranger/authorization/hadoop/config/RangerAuditConfig.java
>  PRE-CREATION 
>   
> agents-common/src/main/java/org/apache/ranger/authorization/hadoop/config/RangerConfiguration.java
>  481fbcc4d 
>   
> agents-common/src/main/java/org/apache/ranger/authorization/hadoop/config/RangerPluginConfig.java
>  PRE-CREATION 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/audit/RangerDefaultAuditHandler.java
>  6b6e91dec 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/conditionevaluator/RangerScriptExecutionContext.java
>  0c078a845 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerAbstractContextEnricher.java
>  737ce0424 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerAdminTagRetriever.java
>  f766e0539 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerTagEnricher.java
>  4e56f5c09 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPluginContext.java
>  df21c5d59 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java
>  45b223d80 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
>  065120f84 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java
>  689985c23 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java
>  84696059a 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerResourceAccessCache.java
>  55f3a0449 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerResourceAccessCacheImpl.java
>  bde65b87f 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java
>  df09b15b2 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java
>  65734f482 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractServiceStore.java
>  82c769f3f 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java
>  c7ea948c9 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/util/PolicyRefresher.java
>  7ec8495bc 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java
>  fe8712db2 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTUtils.java
>  52cea4d74 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/util/RangerResourceTrie.java
>  1afd07d1a 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRolesProvider.java
>  38ba6cfb4 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/util/ServiceDefUtil.java 
> f82f65f10 
>   
> agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyEngine.java
>  83bbffc48 
>   
> hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessor.java
>  311d3eb0a 
>   
> hbase-agent/src/test/java/org/apache/ranger/authorization/hbase/RangerAdminClientImpl.java
>  48c041000 
>   
> hbase-agent/src/test/java/org/apache/ranger/authorization/hbase/TestPolicyEngine.java
>  919920dff 
>   
> hdfs-agent/src/main/java/org/apache/ranger/authorization/hadoop/RangerHdfsAuthorizer.java
>  52df2db94 
>   
> hdfs-agent/src/test/java/org/apache/ranger/services/hdfs/RangerAdminClientImpl.java
>  3f8d1c8db 
>   
> hdfs-agent/src/test/java/org/apache/ranger/services/hdfs/RangerHdfsAuthorizerTest.java
>  ac53dcf2c 
>   
> hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuditHandler.java
>  765da59b5 
>   
> hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java
>  f10bde425 
>   
> hive-agent/src/test/java/org/apache/ranger/services/hive/RangerAdminClientImpl.java
>  94400161e 
>   
> knox-agent/src/main/java/org/apache/ranger/admin/client/RangerAdminJersey2RESTClient.java
>  0fafa6e4b 
>   
> knox-agent/src/main/java/org/apache/ranger/authorization/knox/KnoxRangerPlugin.java
>  94a47b331 
>   
> knox-agent/src/test/java/org/apache/ranger/services/knox/RangerAdminClientImpl.java
>  82444d5e1 
>   
> plugin-atlas/src/main/java/org/apache/ranger/authorization/atlas/authorizer/RangerAtlasAuthorizer.java
>  1c5658912 
>   
> plugin-elasticsearch/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAuthorizer.java
>  31de63195 
>   
> plugin-kafka/src/main/java/org/apache/ranger/services/kafka/RangerServiceKafka.java
>  37ac3e494 
>   
> plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/RangerAdminClientImpl.java
>  ad3ff0d00 
>   
> plugin-kms/src/main/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizer.java
>  aab46399e 
>   
> plugin-kms/src/main/java/org/apache/ranger/services/kms/RangerServiceKMS.java 
> 579f6396b 
>   
> plugin-kms/src/test/java/org/apache/ranger/authorization/kms/authorizer/RangerAdminClientImpl.java
>  6290f36c2 
>   
> plugin-kylin/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java
>  ed935f8dc 
>   
> plugin-kylin/src/test/java/org/apache/ranger/authorization/kylin/authorizer/RangerAdminClientImpl.java
>  2e361c036 
>   
> plugin-solr/src/main/java/org/apache/ranger/authorization/solr/authorizer/RangerSolrAuditHandler.java
>  96f6d4951 
>   
> plugin-solr/src/main/java/org/apache/ranger/authorization/solr/authorizer/RangerSolrAuthorizer.java
>  46d0f6605 
>   
> plugin-sqoop/src/main/java/org/apache/ranger/authorization/sqoop/authorizer/RangerSqoopAuthorizer.java
>  d099f0067 
>   
> plugin-sqoop/src/test/java/org/apache/ranger/authorization/sqoop/authorizer/RangerAdminClientImpl.java
>  758d68106 
>   
> plugin-yarn/src/main/java/org/apache/ranger/authorization/yarn/authorizer/RangerYarnAuthorizer.java
>  4f186193a 
>   
> ranger-examples/plugin-sampleapp/src/main/java/org/apache/ranger/examples/sampleapp/RangerAuthorizer.java
>  c06894e31 
>   
> ranger-tools/src/main/java/org/apache/ranger/policyengine/PerfTestEngine.java 
> df31a6175 
>   
> ranger-tools/src/main/java/org/apache/ranger/policyengine/RangerPluginPerfTester.java
>  4fc6655aa 
>   
> ranger-tools/src/main/java/org/apache/ranger/policyengine/RangerPolicyenginePerfTester.java
>  53e5cf023 
>   security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java 
> abb1b1013 
>   security-admin/src/main/java/org/apache/ranger/biz/RoleDBStore.java 
> 213639a33 
>   security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java 
> 0ab733c65 
>   security-admin/src/main/java/org/apache/ranger/biz/TagDBStore.java 
> 0e33298db 
>   
> security-admin/src/main/java/org/apache/ranger/common/RangerAdminTagEnricher.java
>  ad8d43bcf 
>   security-admin/src/main/java/org/apache/ranger/common/RangerRoleCache.java 
> b0bd427d7 
>   
> security-admin/src/main/java/org/apache/ranger/common/RangerServicePoliciesCache.java
>  8942b4ef7 
>   
> security-admin/src/main/java/org/apache/ranger/common/RangerServiceTagsCache.java
>  22a92b206 
>   security-admin/src/main/java/org/apache/ranger/patch/BaseLoader.java 
> 66f8e7c56 
>   
> security-admin/src/main/java/org/apache/ranger/patch/PatchForTagServiceDefUpdate_J10028.java
>  b113bd49e 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 
> 3d8a32977 
>   
> security-admin/src/main/java/org/apache/ranger/service/RangerServiceDefService.java
>  3f7068ce7 
>   
> storm-agent/src/main/java/org/apache/ranger/authorization/storm/StormRangerPlugin.java
>  111083c4d 
>   
> storm-agent/src/test/java/org/apache/ranger/authorization/storm/RangerAdminClientImpl.java
>  895ad4ad2 
>   
> tagsync/src/main/java/org/apache/ranger/tagsync/sink/tagadmin/TagAdminRESTSink.java
>  062c5e307 
>   
> ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java
>  27506d17a 
>   
> ugsync/src/main/java/org/apache/ranger/unixusersync/process/RangerUgSyncRESTClient.java
>  52b7f62fb 
> 
> 
> Diff: https://reviews.apache.org/r/71759/diff/1/
> 
> 
> Testing
> -------
> 
> - unit tests passed
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>

Reply via email to