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/06/02 12:41:07 UTC

[4/6] jena git commit: Note that TopN limit must be less than the spill-to-disk for Distinct.

Note that TopN limit must be less than the spill-to-disk for Distinct.

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

Branch: refs/heads/master
Commit: a5f3c8e2254d3594a56c01c945b257cbce7d48c8
Parents: 244ebeb
Author: Andy Seaborne <an...@apache.org>
Authored: Tue Jun 2 11:34:08 2015 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Tue Jun 2 11:34:08 2015 +0100

----------------------------------------------------------------------
 .../org/apache/jena/sparql/algebra/optimize/TransformTopN.java     | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/a5f3c8e2/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/TransformTopN.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/TransformTopN.java b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/TransformTopN.java
index 56c2f08..2c49133 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/TransformTopN.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/optimize/TransformTopN.java
@@ -40,6 +40,8 @@ import org.apache.jena.sparql.util.Symbol ;
  */
 public class TransformTopN extends TransformCopy {
 
+    // This must be less than ARQ.spillToDiskThreshold.
+    // Otherwise DISTINCT ends up reordering. 
 	private static final int defaultTopNSortingThreshold = 1000;
 	public static final Symbol externalSortBufferSize = ARQConstants.allocSymbol("topNSortingThreshold") ;