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