[jira] [Commented] (RANGER-3302) GetAuthzEvents should check if auditProvider is null

["Haihui Xu (Jira)" <ji...@apache.org>]

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