Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13663 )
Change subject: IMPALA-8671: Do not re-create RangerAuthorizationChecker instance on catalog update ...................................................................... IMPALA-8671: Do not re-create RangerAuthorizationChecker instance on catalog update This patch fixes the issue where RangerAuthorizationChecker instance gets created on every catalog update, which can impact the performance due to the need to re-initialize the Ranger plugin. Unlike SentryAuthorizationChecker which is stateful (it uses AuthorizationPolicy), RangerAuthorizationChecker is stateless and we can reuse the same instance of it. Testing: - Ran FE tests - Ran E2E authorization tests Change-Id: Iabd5c1b5c7cafc7a03681def0c0f6f775d690c41 Reviewed-on: http://gerrit.cloudera.org:8080/13663 Reviewed-by: Todd Lipcon <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/authorization/ranger/RangerAuthorizationFactory.java 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Todd Lipcon: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/13663 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iabd5c1b5c7cafc7a03681def0c0f6f775d690c41 Gerrit-Change-Number: 13663 Gerrit-PatchSet: 5 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
