[ranger] branch master updated: RANGER-4478: Incorrect trie updates when processing deltas - Part 2

[abhay]

This is an automated email from the ASF dual-hosted git repository.

abhay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/master by this push:
     new 36ce62eab RANGER-4478: Incorrect trie updates when processing deltas - 
Part 2
36ce62eab is described below

commit 36ce62eabbcc38112b15e376411fb053ef8d2ed9
Author: Abhay Kulkarni <ab...@apache.org>
AuthorDate: Mon Oct 23 13:18:12 2023 -0700

    RANGER-4478: Incorrect trie updates when processing deltas - Part 2
---
 .../java/org/apache/ranger/plugin/policyengine/RangerResourceTrie.java | 3 ---
 1 file changed, 3 deletions(-)

diff --git 
a/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceTrie.java
 
b/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceTrie.java
index 61b6a4357..d95da7c50 100644
--- 
a/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceTrie.java
+++ 
b/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceTrie.java
@@ -553,7 +553,6 @@ public class RangerResourceTrie<T extends 
RangerResourceEvaluator> {
 
             builderThreads.get(index).add(resource, isRecursive, evaluator);
         } else {
-            currentRoot.undoSetup();
             currentRoot.addWildcardEvaluator(evaluator);
         }
 
@@ -570,7 +569,6 @@ public class RangerResourceTrie<T extends 
RangerResourceEvaluator> {
         }
 
         if(isWildcard || isRecursive) {
-            curr.undoSetup();
             curr.addWildcardEvaluator(evaluator);
         } else {
             curr.addEvaluator(evaluator);
@@ -1301,7 +1299,6 @@ public class RangerResourceTrie<T extends 
RangerResourceEvaluator> {
 
         private void removeEvaluatorFromSubtree(U evaluator) {
             if (CollectionUtils.isNotEmpty(wildcardEvaluators) && 
wildcardEvaluators.contains(evaluator)) {
-                undoSetup();
                 removeWildcardEvaluator(evaluator);
             } else {
                 removeEvaluator(evaluator);