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 2013/04/13 22:28:28 UTC

svn commit: r1467698 - /jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/TestTransformQuads.java

Author: andy
Date: Sat Apr 13 20:28:28 2013
New Revision: 1467698

URL: http://svn.apache.org/r1467698
Log:
@ignore optimization tests that break when improvements made for more common cases (partial work on JENA-432)

Modified:
    jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/TestTransformQuads.java

Modified: jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/TestTransformQuads.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/TestTransformQuads.java?rev=1467698&r1=1467697&r2=1467698&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/TestTransformQuads.java (original)
+++ jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/TestTransformQuads.java Sat Apr 13 20:28:28 2013
@@ -20,6 +20,7 @@ package com.hp.hpl.jena.sparql.algebra;
 
 import org.apache.jena.atlas.junit.BaseTest ;
 import org.apache.jena.atlas.lib.StrUtils ;
+import org.junit.Ignore ;
 import org.junit.Test ;
 
 import com.hp.hpl.jena.query.Query ;
@@ -59,6 +60,7 @@ public class TestTransformQuads extends 
                                         "     (quadpattern (quad ?g ?s1 ?p1 ?o1))))" 
                                         ) ; }
     // Filters
+    @Ignore("This optimization is suspect - improving more common filter-equality usage stops this case")
     @Test public void quads10() { test ("{ GRAPH ?g { ?s ?p ?o FILTER (str(?g) = 'graphURI') } }", 
                                         "(assign ((?g ?*g0))" +
                                         "   (filter (= (str ?g) 'graphURI')" +
@@ -71,6 +73,7 @@ public class TestTransformQuads extends 
                                         ) ; }
     
     // Nested and filter
+    @Ignore("This optimization is suspect - improving more common filter-equality usage stops this case")
     @Test public void quads20() { test ("{ GRAPH ?g { ?s ?p ?o GRAPH ?g1 { ?s1 ?p1 ?o1 FILTER (str(?g) = 'graphURI') } } }",
                                         "(assign ((?g ?*g0))" +
                                         "   (sequence" +
@@ -79,6 +82,7 @@ public class TestTransformQuads extends 
                                         "       (quadpattern (quad ?g1 ?s1 ?p1 ?o1)))))"
                                         ) ; }
     
+    @Ignore("This optimization is suspect - improving more common filter-equality usage stops this case")
     @Test public void quads21() { test ("{ GRAPH ?g { ?s ?p ?o GRAPH ?g1 { ?s1 ?p1 ?o1 FILTER (str(?g1) = 'graphURI') } } }",
                                         "(sequence" +
                                         "   (quadpattern (quad ?g ?s ?p ?o))" +
@@ -88,6 +92,7 @@ public class TestTransformQuads extends 
                                        ) ; }
     
     // Tricky pattern ... twice.
+    @Ignore("This optimization is suspect - improving more common filter-equality usage stops this case")
     @Test public void quads30() { test ( "{ GRAPH ?g { ?s ?p ?o FILTER (str(?g) = 'graphURI') } " +
                                          "  GRAPH ?g { ?s ?p ?o FILTER (str(?g) = 'graphURI') } }",
                                          "(sequence" +