[ https://issues.apache.org/jira/browse/RANGER-2654?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Madhan Neethiraj updated RANGER-2654: ------------------------------------- Attachment: RANGER-2654-3.patch > refactor RangerBasePlugins to remove static servicePluginsMap > -------------------------------------------------------------- > > Key: RANGER-2654 > URL: https://issues.apache.org/jira/browse/RANGER-2654 > Project: Ranger > Issue Type: Improvement > Components: plugins > Reporter: Madhan Neethiraj > Assignee: Madhan Neethiraj > Priority: Major > Fix For: master > > Attachments: RANGER-2654-3.patch, RANGER-2654.patch > > > Currently RangerBasePlugin has a static member {{servicePluginsMap}}, which > is used in following cases: > 1. to notify plugin of change in auth-context, like updates to tags > 2. to get audit provider for the plugin, from > RangerDefaultAuditHandler.logAuthzAudit() > #1 can be addressed by moving the notification details to RangerAuthContext > #2 can be addressed by getting the audit provider from the global > audit-provider factory instance. This move (of static) from > plugin->audit-factory would enable multiple plugin instances to use same > audit provider. > In addition, much of the state information in RangerBasePlugin (like > serviceName, serviceType, appId, etc.) should be moved to another class, like > RangerPluginConfig. -- This message was sent by Atlassian Jira (v8.3.4#803005)