[
https://issues.apache.org/jira/browse/RANGER-1678?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
peng.jianhua updated RANGER-1678:
---------------------------------
Attachment: 0001-RANGER-1678-In-different-places-to-achieve-the-same-.patch
> In different places to achieve the same function using repeat codes, new
> issue is perhaps generated when these functions are modified.
> --------------------------------------------------------------------------------------------------------------------------------------
>
> Key: RANGER-1678
> URL: https://issues.apache.org/jira/browse/RANGER-1678
> Project: Ranger
> Issue Type: Bug
> Components: plugins
> Reporter: peng.jianhua
> Assignee: peng.jianhua
> Labels: newbie, patch
> Fix For: master
>
> Attachments:
> 0001-RANGER-1678-In-different-places-to-achieve-the-same-.patch
>
>
> In agents-audit module, there are following two places of writing audit logs
> to solr:
> 1. org.apache.ranger.audit.provider.solr.SolrAuditProvider.
> 2. org.apache.ranger.audit.destination.SolrAuditDestination.
> Above classes use the same method( "MiscUtil.executePrivilegedAction") to
> send the audit logs to solr. Codes is as following
> final UpdateResponse response = MiscUtil.executePrivilegedAction(new
> PrivilegedExceptionAction<UpdateResponse>() {
> @Override
> public UpdateResponse run() throws Exception {
> return solrClient.add(docs);
> }
> });
> We should extract the common method to let our codes more cleaner and reduce
> the possibility of new issue.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)