[Bigdata-commit] SF.net SVN: bigdata:[7443] branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src

[<jer...@us...>]

Revision: 7443
          http://bigdata.svn.sourceforge.net/bigdata/?rev=7443&view=rev
Author:   jeremy_carroll
Date:     2013-10-09 20:41:28 +0000 (Wed, 09 Oct 2013)
Log Message:
-----------
Fix for 725: optimize exists inside subqueries. Enable test for this.

Modified Paths:
--------------
    branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/java/com/bigdata/rdf/sparql/ast/optimizers/ASTExistsOptimizer.java
    branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/test/com/bigdata/rdf/sparql/ast/eval/TestAll.java

Modified: branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/java/com/bigdata/rdf/sparql/ast/optimizers/ASTExistsOptimizer.java
===================================================================
--- branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/java/com/bigdata/rdf/sparql/ast/optimizers/ASTExistsOptimizer.java	2013-10-09 18:42:30 UTC (rev 7442)
+++ branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/java/com/bigdata/rdf/sparql/ast/optimizers/ASTExistsOptimizer.java	2013-10-09 20:41:28 UTC (rev 7443)
@@ -180,6 +180,12 @@
 
             }
 
+            if (child instanceof SubqueryRoot) {
+                // Recursion.
+                SubqueryRoot subquery = (SubqueryRoot)child;
+				rewrite(sa, exogenousVars, subquery, subquery.getWhereClause() );
+            }
+
         }
 
     }

Modified: branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/test/com/bigdata/rdf/sparql/ast/eval/TestAll.java
===================================================================
--- branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/test/com/bigdata/rdf/sparql/ast/eval/TestAll.java	2013-10-09 18:42:30 UTC (rev 7442)
+++ branches/BIGDATA_RELEASE_1_3_0/bigdata-rdf/src/test/com/bigdata/rdf/sparql/ast/eval/TestAll.java	2013-10-09 20:41:28 UTC (rev 7443)
@@ -156,6 +156,8 @@
          * Tests corresponding to various trouble tickets.
          */
         suite.addTestSuite(TestTickets.class);
+        
+        suite.addTestSuite(TestSubSelectFilterExist725.class);
         suite.addTestSuite(TestTwoPropertyPaths734.class);
         
         // test suite for inline constraints: GT, LT, GTE, LTE

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.