You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ab...@apache.org on 2023/10/23 22:49:37 UTC
[ranger] branch master updated: RANGER-4478: Incorrect trie updates when processing deltas - Part 2
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);