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