Document Project: http://git-wip-us.apache.org/repos/asf/jena/repo Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/5ed532b4 Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/5ed532b4 Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/5ed532b4
Branch: refs/heads/master Commit: 5ed532b48758bd58c2703c0fe55f2ac593a6c0bd Parents: eeb4e4a Author: Andy Seaborne <[email protected]> Authored: Tue Sep 23 14:47:09 2014 +0100 Committer: Andy Seaborne <[email protected]> Committed: Tue Sep 23 14:47:09 2014 +0100 ---------------------------------------------------------------------- .../sparql/algebra/optimize/TransformPathFlattern.java | 12 +++++++++--- .../algebra/optimize/TransformPathFlatternStd.java | 1 + 2 files changed, 10 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jena/blob/5ed532b4/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlattern.java ---------------------------------------------------------------------- diff --git a/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlattern.java b/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlattern.java index 52e236b..af5e1d9 100644 --- a/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlattern.java +++ b/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlattern.java @@ -25,9 +25,15 @@ import com.hp.hpl.jena.sparql.core.PathBlock ; import com.hp.hpl.jena.sparql.path.PathCompiler ; import com.hp.hpl.jena.sparql.path.PathLib ; -/** Perform the equivalent of the trasnactions in the SPARQL 1.1 spec. - * ARQ regards path transformation as an optimization. - * ARQ does not execute the exact transformation as per spec as there are better ways to do it. +/** + * Perform the equivalent of the transactions in the SPARQL 1.1 spec. ARQ + * regards path transformation as an optimization. ARQ does not execute the + * exact transformation as per spec as there are better ways to do it for ARQ. + * For example, + * <ul> + * <li>Path seq -> BGPs or a (sequence) + * <li>"|" is not expanded into a union. + * </ul> */ public class TransformPathFlattern extends TransformCopy http://git-wip-us.apache.org/repos/asf/jena/blob/5ed532b4/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlatternStd.java ---------------------------------------------------------------------- diff --git a/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlatternStd.java b/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlatternStd.java index 3433345..163aa25 100644 --- a/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlatternStd.java +++ b/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformPathFlatternStd.java @@ -37,6 +37,7 @@ import com.hp.hpl.jena.sparql.core.VarAlloc ; import com.hp.hpl.jena.sparql.path.* ; /** The path transformation step exactly as per the SPARQL 1.1 spec. + * i.e. joins triples rather creatign BGPs. * It does not produce very nice execution structures so ARQ uses * a functional equivalent, but different, transformation. */
