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 2015/09/08 19:10:40 UTC

[08/20] jena git commit: Minor

Minor


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

Branch: refs/heads/master
Commit: f25095c47d8d5209a9854631b38d7f2acfcb8c55
Parents: 24e4c1e
Author: Andy Seaborne <an...@apache.org>
Authored: Tue Sep 8 14:18:21 2015 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Tue Sep 8 18:08:12 2015 +0100

----------------------------------------------------------------------
 .../jena/sparql/engine/join/QueryIterNestedLoopJoin.java  | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/f25095c4/jena-arq/src/main/java/org/apache/jena/sparql/engine/join/QueryIterNestedLoopJoin.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/engine/join/QueryIterNestedLoopJoin.java b/jena-arq/src/main/java/org/apache/jena/sparql/engine/join/QueryIterNestedLoopJoin.java
index 0208697..19684c3 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/engine/join/QueryIterNestedLoopJoin.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/engine/join/QueryIterNestedLoopJoin.java
@@ -32,7 +32,7 @@ import org.apache.jena.sparql.engine.iterator.QueryIter2;
  * Nested Loop Join (materializing on the left, streaming on the right)
  * A simple, dependable join.
  * <p>
- * See {@link Join#innerLoopJoinBasic} for a very simple implementation for 
+ * See {@link Join#nestedLoopLeftJoinBasic} for a very simple implementation for 
  * testing purposes only. 
  */
 public class QueryIterNestedLoopJoin extends QueryIter2 {
@@ -50,9 +50,8 @@ public class QueryIterNestedLoopJoin extends QueryIter2 {
 
     public QueryIterNestedLoopJoin(QueryIterator left, QueryIterator right, ExecutionContext cxt) {
         super(left, right, cxt);
-        List<Binding> rowsLeftList = Iter.toList(left);
-        leftRows = rowsLeftList;
-        s_countLHS = rowsLeftList.size();
+        leftRows = Iter.toList(left);
+        s_countLHS = leftRows.size();
         this.right = right;
     }
 
@@ -98,9 +97,6 @@ public class QueryIterNestedLoopJoin extends QueryIter2 {
                 if ( r != null ) {
                     s_countResults++;
                     return r;
-                } else {
-                    // XXX LeftJoin
-                    //return rowLeft ;
                 }
             }
             // Nothing more for this rowRight.