[RANGER-3302] GetAuthzEvents should check if auditProvider is null

• Type: Improvement
• Status: Patch Available
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 2.1.0
• Fix Version/s: 3.0.0
• Component/s: audit
• Labels: patch performance
• Assignee: Haihui Xu
• Reporter: Haihui Xu
• Due: 06/Jun/21
• Created: 01/Jun/21 01:42
• Updated: 01/Sep/21 09:17

Description

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