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?


---

Reply via email to