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 2017/06/01 10:58:26 UTC

[1/2] jena git commit: Better debug print output

Repository: jena
Updated Branches:
  refs/heads/master 08abdebae -> 41ebfe016


Better debug print output


Project: http://git-wip-us.apache.org/repos/asf/jena/repo
Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/c9c6a849
Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/c9c6a849
Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/c9c6a849

Branch: refs/heads/master
Commit: c9c6a849dddf025c0ef280df7a34828bf398a0ad
Parents: 08abdeb
Author: Andy Seaborne <an...@apache.org>
Authored: Thu Jun 1 11:58:05 2017 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Thu Jun 1 11:58:05 2017 +0100

----------------------------------------------------------------------
 .../java/org/apache/jena/sparql/engine/main/JoinClassifier.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/c9c6a849/jena-arq/src/main/java/org/apache/jena/sparql/engine/main/JoinClassifier.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/engine/main/JoinClassifier.java b/jena-arq/src/main/java/org/apache/jena/sparql/engine/main/JoinClassifier.java
index 00b06ea..b90d56e 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/engine/main/JoinClassifier.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/engine/main/JoinClassifier.java
@@ -33,8 +33,6 @@ public class JoinClassifier
     static /*final*/ public  boolean print = false ;
 
     static public boolean isLinear(OpJoin join) {
-        if ( print )
-            System.err.println(join) ;
         return isLinear(join.getLeft(), join.getRight()) ;
     }
 
@@ -83,7 +81,9 @@ public class JoinClassifier
     // Check left can stream into right
     static private boolean check(Op leftOp, Op rightOp) {
         if ( print ) {
+            System.err.println("Left::");
             System.err.println(leftOp) ;
+            System.err.println("Right::");
             System.err.println(rightOp) ;
         }
 


[2/2] jena git commit: Comments and format fixes

Posted by an...@apache.org.
Comments and format fixes


Project: http://git-wip-us.apache.org/repos/asf/jena/repo
Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/41ebfe01
Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/41ebfe01
Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/41ebfe01

Branch: refs/heads/master
Commit: 41ebfe01679a4bee830f53fe54bd9352e4f5a278
Parents: c9c6a84
Author: Andy Seaborne <an...@apache.org>
Authored: Thu Jun 1 11:58:22 2017 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Thu Jun 1 11:58:22 2017 +0100

----------------------------------------------------------------------
 .../java/org/apache/jena/sparql/algebra/optimize/Optimize.java   | 4 ++--
 .../src/main/java/org/apache/jena/sparql/core/Substitute.java    | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/41ebfe01/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/Optimize.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/Optimize.java b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/Optimize.java
index 73ddf6d..8b6b6fe 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/Optimize.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/Optimize.java
@@ -68,12 +68,12 @@ public class Optimize
     }
 
     /** Set the global optimizer factory to one that does nothing.
-     * Applications probably wany {@link #basicOptimizer}  */
+     * Applications probably want {@link #basicOptimizer}  */
     public static void noOptimizer() {
         setFactory(noOptimizationFactory) ;
     }
 
-    /** Set the global optimizer factory to one that only does property functions and scaoped variables.
+    /** Set the global optimizer factory to one that only does property functions and scoped variables.
      * @see #minimalOptimizationFactory
      */
     public static void basicOptimizer() {

http://git-wip-us.apache.org/repos/asf/jena/blob/41ebfe01/jena-arq/src/main/java/org/apache/jena/sparql/core/Substitute.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/core/Substitute.java b/jena-arq/src/main/java/org/apache/jena/sparql/core/Substitute.java
index e27b4b1..eaccd32 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/core/Substitute.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/core/Substitute.java
@@ -124,8 +124,8 @@ public class Substitute {
         if ( propFuncArg.isNode() ) {
             Node n = propFuncArg.getArg();
             if ( !Var.isVar(n) )
-                                // Not a Var, no substitute needed.
-                                return propFuncArg;
+                // Not a Var, no substitute needed.
+                return propFuncArg;
             return new PropFuncArg(substitute(propFuncArg.getArg(), binding));
         }