Skip to content

Commit bfa7500

Browse files
GH-4769 Hiding variables out of scope is only for the filter condition.
Also removed a debug aid.
1 parent 672e4f9 commit bfa7500

1 file changed

Lines changed: 4 additions & 13 deletions

File tree

core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/impl/DefaultEvaluationStrategy.java

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -633,21 +633,12 @@ protected QueryEvaluationStep prepare(Service service, QueryEvaluationContext co
633633

634634
protected QueryEvaluationStep prepare(Filter node, QueryEvaluationContext context) throws QueryEvaluationException {
635635

636-
final FilterIterator.RetainedVariableFilteredQueryEvaluationContext context2 = new FilterIterator.RetainedVariableFilteredQueryEvaluationContext(
637-
node, context);
638-
QueryEvaluationStep arg = precompile(node.getArg(), context2);
636+
QueryEvaluationStep arg = precompile(node.getArg(), context);
639637
QueryValueEvaluationStep ves;
640638
try {
641-
var ves2 = precompile(node.getCondition(), context2);
642-
ves = new QueryValueEvaluationStep() {
643-
644-
@Override
645-
public Value evaluate(BindingSet bindings) throws QueryEvaluationException {
646-
assert (node != null);
647-
return ves2.evaluate(bindings);
648-
}
649-
650-
};
639+
final FilterIterator.RetainedVariableFilteredQueryEvaluationContext context2 = new FilterIterator.RetainedVariableFilteredQueryEvaluationContext(
640+
node, context);
641+
ves = precompile(node.getCondition(), context2);
651642
} catch (QueryEvaluationException e) {
652643
// If we have a failed compilation we always return false.
653644
// Which means empty. so let's short circuit that.

0 commit comments

Comments
 (0)