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 2016/12/12 08:47:53 UTC

jena git commit: Correct the weighting of S?O.

Repository: jena
Updated Branches:
  refs/heads/master 098206dbf -> f99f2f29e


Correct the weighting of S?O.

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

Branch: refs/heads/master
Commit: f99f2f29e33788b637f6c3fe806ac4ec8f3a0c39
Parents: 098206d
Author: Andy Seaborne <an...@apache.org>
Authored: Mon Dec 12 08:47:24 2016 +0000
Committer: Andy Seaborne <an...@apache.org>
Committed: Mon Dec 12 08:47:24 2016 +0000

----------------------------------------------------------------------
 .../apache/jena/sparql/engine/optimizer/reorder/ReorderFixed.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/f99f2f29/jena-arq/src/main/java/org/apache/jena/sparql/engine/optimizer/reorder/ReorderFixed.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/engine/optimizer/reorder/ReorderFixed.java b/jena-arq/src/main/java/org/apache/jena/sparql/engine/optimizer/reorder/ReorderFixed.java
index 4fddc7d..2905787 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/engine/optimizer/reorder/ReorderFixed.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/engine/optimizer/reorder/ReorderFixed.java
@@ -69,7 +69,6 @@ public class ReorderFixed extends ReorderTransformationSubstitution {
     static { init() ; }
     
     private static void init() {
-        //ype = Item.createNode(NodeConst.nodeRDFType) ;
         // rdf:type can be a bad choice e.g rdf:type rdf:Resource
         // with inference enabled.
         // Weight use of rdf:type worse then the general pattern
@@ -87,7 +86,7 @@ public class ReorderFixed extends ReorderTransformationSubstitution {
 
         matcher.addPattern(new Pattern(2, TERM, TERM, VAR)) ;                   // SP?
         matcher.addPattern(new Pattern(3, VAR, TERM, TERM)) ;                   // ?PO
-        matcher.addPattern(new Pattern(2, TERM, TERM, TERM)) ;                  // S?O
+        matcher.addPattern(new Pattern(2, TERM, VAR, TERM)) ;                   // S?O
 
         matcher.addPattern(new Pattern(10, TERM, VAR, VAR)) ;                   // S??
         matcher.addPattern(new Pattern(20, VAR, VAR, TERM)) ;                   // ??O