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/08/27 20:50:48 UTC

[1/3] jena git commit: Remove stray blank line.

Repository: jena
Updated Branches:
  refs/heads/master 384f1da08 -> dabdb5334


Remove stray blank line.

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

Branch: refs/heads/master
Commit: b28dab163a61860dd232e0588dcaa1a95ef9803c
Parents: 384f1da
Author: Andy Seaborne <an...@apache.org>
Authored: Sat Aug 27 17:37:04 2016 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Sat Aug 27 17:37:04 2016 +0100

----------------------------------------------------------------------
 .../main/java/org/apache/jena/sparql/algebra/optimize/Optimize.java | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/b28dab16/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 0e7e45a..8cd61d6 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
@@ -242,7 +242,6 @@ public class Optimize implements Rewrite
         if ( context.isTrue(ARQ.optFilterInequality) )
             op = apply("Filter Inequality", new TransformFilterInequality(), op);
         
-        
         // Promote table empty as late as possible since this will only be produced by other 
         // optimizations and never directly from algebra generation
         if ( context.isTrueOrUndef(ARQ.optPromoteTableEmpty) )


[2/3] jena git commit: Set the max journal queue length default to 100

Posted by an...@apache.org.
Set the max journal queue length default to 100


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

Branch: refs/heads/master
Commit: 3f5e15a0bb94e726d640fed703170ca1706c476c
Parents: b28dab1
Author: Andy Seaborne <an...@apache.org>
Authored: Sat Aug 27 21:49:57 2016 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Sat Aug 27 21:49:57 2016 +0100

----------------------------------------------------------------------
 .../java/org/apache/jena/tdb/transaction/TransactionManager.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/3f5e15a0/jena-tdb/src/main/java/org/apache/jena/tdb/transaction/TransactionManager.java
----------------------------------------------------------------------
diff --git a/jena-tdb/src/main/java/org/apache/jena/tdb/transaction/TransactionManager.java b/jena-tdb/src/main/java/org/apache/jena/tdb/transaction/TransactionManager.java
index 76a36b9..60c4d88 100644
--- a/jena-tdb/src/main/java/org/apache/jena/tdb/transaction/TransactionManager.java
+++ b/jena-tdb/src/main/java/org/apache/jena/tdb/transaction/TransactionManager.java
@@ -77,7 +77,7 @@ public class TransactionManager
      * before deciding to pause and flush as soon as possible.
      * -1 means "off".   
      */
-    public static /*final*/ int MaxQueueThreshold = 250 ;
+    public static /*final*/ int MaxQueueThreshold = 100 ;
     
     private static int setQueueBatchSize() {
         if ( SystemTDB.is64bitSystem )


[3/3] jena git commit: Comments and deprecation for Opfilter.filter

Posted by an...@apache.org.
Comments and deprecation for Opfilter.filter


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

Branch: refs/heads/master
Commit: dabdb5334648b022f70a19f3d67d720458e05a39
Parents: 3f5e15a
Author: Andy Seaborne <an...@apache.org>
Authored: Sat Aug 27 21:50:31 2016 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Sat Aug 27 21:50:31 2016 +0100

----------------------------------------------------------------------
 .../apache/jena/sparql/algebra/op/OpFilter.java   | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/dabdb533/jena-arq/src/main/java/org/apache/jena/sparql/algebra/op/OpFilter.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/op/OpFilter.java b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/op/OpFilter.java
index 83b384d..4b01b8a 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/op/OpFilter.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/op/OpFilter.java
@@ -32,16 +32,20 @@ public class OpFilter extends Op1
     
     /** Add expression - mutates an existing filter */  
     public static Op filter(Expr expr, Op op) {
-        OpFilter f = filter(op) ;
+        OpFilter f = ensureFilter(op) ;
         f.getExprs().add(expr) ;
         return f ;
     }
 
     /**
      * Ensure that the algebra op is a filter. If the input is a filter, just return that,
-     * else create a filter with no expressions and this as the subOp
+     * else create a filter with no expressions and "this" as the subOp.
+     * @apiNote
+     * This operation assumes the caller is going to add expressions. 
+     * Filters without any expressions are discouraged.
+     * Consider collecting the expressions together first and using {@link #filterBy}. 
      */
-    public static OpFilter filter(Op op) {
+    public static OpFilter ensureFilter(Op op) {
         if ( op instanceof OpFilter )
             return (OpFilter)op ;
         else
@@ -54,6 +58,10 @@ public class OpFilter extends Op1
         return filterBy(exprs, op) ;
     }
     
+    /** @deprecated Use {@link #ensureFilter} */
+    @Deprecated
+    public static OpFilter filter(Op op) { return ensureFilter(op) ; }
+    
     /** Combine an ExprList with an Op so that the expressions filter the Op.
      * If the exprs are empty, return the Op.  
      * If the op is already a OpFilter, merge the expressions into the filters existintg expressions.  
@@ -62,7 +70,7 @@ public class OpFilter extends Op1
     public static Op filterBy(ExprList exprs, Op op) {
         if ( exprs == null || exprs.isEmpty() )
             return op ;
-        OpFilter f = filter(op) ;
+        OpFilter f = ensureFilter(op) ;
         f.getExprs().addAll(exprs) ;
         return f ;
     }
@@ -71,7 +79,7 @@ public class OpFilter extends Op1
      * If subOp is a filter, combine expressions (de-layer).  
      */
     public static OpFilter filterAlways(ExprList exprs, Op subOp) {
-        OpFilter f = filter(subOp) ;
+        OpFilter f = ensureFilter(subOp) ;
         f.getExprs().addAll(exprs) ;
         return f ;
     }