[ 
https://issues.apache.org/jira/browse/RANGER-3302?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17355451#comment-17355451
 ] 

Haihui Xu commented on RANGER-3302:
-----------------------------------

[~madhan] Please review it, thanks very much.

> GetAuthzEvents should check if auditProvider is null
> ----------------------------------------------------
>
>                 Key: RANGER-3302
>                 URL: https://issues.apache.org/jira/browse/RANGER-3302
>             Project: Ranger
>          Issue Type: Improvement
>          Components: audit
>    Affects Versions: 2.1.0
>            Reporter: Haihui Xu
>            Assignee: Haihui Xu
>            Priority: Major
>              Labels: patch, performance
>             Fix For: 2.2.0
>
>         Attachments: RANGER-3302.patch
>
>
> Process the auditlog,need to getAuditProvider and getAuthzEvents, when 
> logAuthzAudit, the code is
>            if (auditProvider == null || !auditProvider.log(auditEvent)) {
>                               MiscUtil.logErrorMessageByInterval(LOG, "fail 
> to log audit event " + auditEvent);
>                       }
> so, when getAuthzEvents should check auditProvider value, when auditProvider 
> is null, there is no need to construct auditEvent, which is time-consuming 
> and will result in the performance decline of service(kakfa broker,namenode, 
> hiveserver2...)



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to