Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/394#discussion_r181393545
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpWalker.java ---
@@ -139,6 +139,7 @@ protected void visitN(OpN op) {
@Override
protected void visitExt(OpExt op) {
before(op) ;
+ super.visitExt(op);
--- End diff --
Not sure about this. It is visiting both the effective op and the real
OpExt. Shoudn't it be the visitor deciding that? ie. visiting
`op.effectiveOp()` is required?
---
