Skip to content

Commit f96b96e

Browse files
lfxyrpignolet
authored andcommitted
RANGER-3377: HDFS plugin performance improvement - conditionally ignore deny and exception conditions
Signed-off-by: Madhan Neethiraj <madhan@apache.org> (cherry picked from commit 79f6cde)
1 parent 643a89c commit f96b96e

File tree

1 file changed

+12
-4
lines changed

1 file changed

+12
-4
lines changed

agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -147,9 +147,17 @@ public void init(RangerPolicy policy, RangerServiceDef serviceDef, RangerPolicyE
147147
denyExceptionEvaluators = Collections.<RangerPolicyItemEvaluator>emptyList();
148148
} else {
149149
allowEvaluators = createPolicyItemEvaluators(policy, serviceDef, options, RangerPolicyItemEvaluator.POLICY_ITEM_TYPE_ALLOW);
150-
denyEvaluators = createPolicyItemEvaluators(policy, serviceDef, options, RangerPolicyItemEvaluator.POLICY_ITEM_TYPE_DENY);
151-
allowExceptionEvaluators = createPolicyItemEvaluators(policy, serviceDef, options, RangerPolicyItemEvaluator.POLICY_ITEM_TYPE_ALLOW_EXCEPTIONS);
152-
denyExceptionEvaluators = createPolicyItemEvaluators(policy, serviceDef, options, RangerPolicyItemEvaluator.POLICY_ITEM_TYPE_DENY_EXCEPTIONS);
150+
151+
if (ServiceDefUtil.getOption_enableDenyAndExceptionsInPolicies(serviceDef)) {
152+
denyEvaluators = createPolicyItemEvaluators(policy, serviceDef, options, RangerPolicyItemEvaluator.POLICY_ITEM_TYPE_DENY);
153+
allowExceptionEvaluators = createPolicyItemEvaluators(policy, serviceDef, options, RangerPolicyItemEvaluator.POLICY_ITEM_TYPE_ALLOW_EXCEPTIONS);
154+
denyExceptionEvaluators = createPolicyItemEvaluators(policy, serviceDef, options, RangerPolicyItemEvaluator.POLICY_ITEM_TYPE_DENY_EXCEPTIONS);
155+
} else {
156+
denyEvaluators = Collections.<RangerPolicyItemEvaluator>emptyList();
157+
allowExceptionEvaluators = Collections.<RangerPolicyItemEvaluator>emptyList();
158+
denyExceptionEvaluators = Collections.<RangerPolicyItemEvaluator>emptyList();
159+
}
160+
153161
}
154162

155163
dataMaskEvaluators = createDataMaskPolicyItemEvaluators(policy, serviceDef, options, policy.getDataMaskPolicyItems());
@@ -1268,4 +1276,4 @@ private List<RangerConditionEvaluator> createRangerPolicyConditionEvaluator(Rang
12681276
return rangerConditionEvaluators;
12691277
}
12701278

1271-
}
1279+
}

0 commit comments

Comments
 (0)