Github user afs commented on a diff in the pull request:
    --- Diff: 
jena-arq/src/main/java/org/apache/jena/sparql/algebra/ ---
    @@ -139,6 +139,7 @@ protected void visitN(OpN op) {
             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