You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2014/09/23 15:50:24 UTC

[08/10] git commit: Document

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 <an...@apache.org>
Authored: Tue Sep 23 14:47:09 2014 +0100
Committer: Andy Seaborne <an...@apache.org>
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.
  */