You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ma...@apache.org on 2025/07/13 22:46:42 UTC
(ranger) branch ranger-2.7 updated: RANGER-5247: updated Ozone authorizer to replace unused instance member with a local variable (#606)
This is an automated email from the ASF dual-hosted git repository.
madhan pushed a commit to branch ranger-2.7
in repository https://gitbox.apache.org/repos/asf/ranger.git
The following commit(s) were added to refs/heads/ranger-2.7 by this push:
new 1ca50f9fb RANGER-5247: updated Ozone authorizer to replace unused instance member with a local variable (#606)
1ca50f9fb is described below
commit 1ca50f9fb53813f43343091e03113e15ccbf8b2c
Author: Madhan Neethiraj <ma...@apache.org>
AuthorDate: Sun Jul 13 15:39:13 2025 -0700
RANGER-5247: updated Ozone authorizer to replace unused instance member with a local variable (#606)
(cherry picked from commit 522e4451348091864c87d9ae7358902ac29f3f68)
---
.../authorization/ozone/authorizer/RangerOzoneAuthorizer.java | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/plugin-ozone/src/main/java/org/apache/ranger/authorization/ozone/authorizer/RangerOzoneAuthorizer.java b/plugin-ozone/src/main/java/org/apache/ranger/authorization/ozone/authorizer/RangerOzoneAuthorizer.java
index 3bd17d52f..3b80da65b 100644
--- a/plugin-ozone/src/main/java/org/apache/ranger/authorization/ozone/authorizer/RangerOzoneAuthorizer.java
+++ b/plugin-ozone/src/main/java/org/apache/ranger/authorization/ozone/authorizer/RangerOzoneAuthorizer.java
@@ -57,7 +57,6 @@ public class RangerOzoneAuthorizer implements IAccessAuthorizer {
private static final Logger LOG = LoggerFactory.getLogger(RangerOzoneAuthorizer.class);
private static volatile RangerBasePlugin rangerPlugin = null;
- RangerDefaultAuditHandler auditHandler = null;
public RangerOzoneAuthorizer() {
RangerBasePlugin plugin = rangerPlugin;
@@ -70,7 +69,7 @@ public RangerOzoneAuthorizer() {
plugin = new RangerBasePlugin("ozone", "ozone");
plugin.init(); // this will initialize policy engine and policy refresher
- auditHandler = new RangerDefaultAuditHandler();
+ RangerDefaultAuditHandler auditHandler = new RangerDefaultAuditHandler();
plugin.setResultProcessor(auditHandler);
rangerPlugin = plugin;
@@ -97,7 +96,9 @@ public boolean checkAccess(IOzoneObj ozoneObject, RequestContext context) {
context.getIp() + ", resourceType = " + ozoneObj.getResourceType() + ")");
}
- if (rangerPlugin == null) {
+ RangerBasePlugin plugin = rangerPlugin;
+
+ if (plugin == null) {
MiscUtil.logErrorMessageByInterval(LOG,
"Authorizer is still not initialized");
return returnValue;
@@ -127,7 +128,7 @@ public boolean checkAccess(IOzoneObj ozoneObject, RequestContext context) {
return returnValue;
}
String action = accessType;
- String clusterName = rangerPlugin.getClusterName();
+ String clusterName = plugin.getClusterName();
RangerAccessRequestImpl rangerRequest = new RangerAccessRequestImpl();
rangerRequest.setUser(ugi.getShortUserName());
@@ -165,7 +166,7 @@ public boolean checkAccess(IOzoneObj ozoneObject, RequestContext context) {
}
try {
- RangerAccessResult result = rangerPlugin
+ RangerAccessResult result = plugin
.isAccessAllowed(rangerRequest);
if (result == null) {
LOG.error("Ranger Plugin returned null. Returning false");