mneethiraj commented on code in PR #555:
URL: https://github.com/apache/ranger/pull/555#discussion_r2060508545
##########
agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java:
##########
@@ -883,14 +883,11 @@ public RangerRoles getRangerRoles() {
}
public Set<RangerRole> getRangerRoleForPrincipal(String principal, String
type) {
- Set<RangerRole> ret = new HashSet<>();
- Set<RangerRole> rangerRoles = null;
+ Set<RangerRole> ret = new HashSet<>();
+ RangerPolicyEngine policyEngine = this.policyEngine;
Review Comment:
@vyommani - which line number are you referring to? Local
variable`policyEngine` (line 887) is referenced later in line 893. Hence it is
necessary to check for null value here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@ranger.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
