You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-commits@db.apache.org by bp...@apache.org on 2007/10/04 17:13:33 UTC

svn commit: r581943 - in /db/derby/code/trunk/java: engine/org/apache/derby/impl/sql/compile/ testing/org/apache/derbyTesting/functionTests/tests/lang/

Author: bpendleton
Date: Thu Oct  4 08:13:32 2007
New Revision: 581943

URL: http://svn.apache.org/viewvc?rev=581943&view=rev
Log:
DERBY-3033: NullPointerExceptions with flattening of NOT EXISTS subqueries

This patch modifies the behavior of queries which contain correlated
subqueries using a NOT EXISTS predicate. For example:

   select ... from a, b where <join criteria> 
       and not exists (select from c where c.col = b.col)

Depending on various conditions, subqueries may be "flattened", which is
a term used to describe an internal rewriting process in which the
subquery is converted into a same-level join with the parent tables.

For many types of queries, such flattening results in multiple tables
equijoined together, and the optimizer may recognize, via transitive
closure of the equijoin predicates, additional opportunities for join
predicates.

However, when a NOT EXISTS table is in the join's from list, join
predicates that mention the NOT EXISTS table cannot be used for such
transitive closure, because these are not equijoin predicates, they
are equi-NON-join predicates: the join succeeds when there are NOT
matching rows in that table.

Prior to this change, Derby might incorrectly infer an equijoin predicate
in such a situation, which could lead to wrong results, or to crashes
during query execution when the join processing attempted to access the
non-existing rows. This change modifies the PredicateList processing to
note when a join predicate includes a NOT EXISTS table, and to omit
that join predicate from transitive closure processing.


Added:
    db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/ExistsWithSubqueriesTest.java   (with props)
Removed:
    db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/ExistsWithSetOpsTest.java
Modified:
    db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/FromList.java
    db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/PredicateList.java
    db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/_Suite.java

Modified: db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/FromList.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/FromList.java?rev=581943&r1=581942&r2=581943&view=diff
==============================================================================
--- db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/FromList.java (original)
+++ db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/FromList.java Thu Oct  4 08:13:32 2007
@@ -1549,6 +1549,42 @@
 	}
 
 	/**
+	 * determine whether this table is NOT EXISTS.
+	 *
+	 * This routine searches for the indicated table number in the fromlist
+	 * and returns TRUE if the table is present in the from list and is 
+	 * marked NOT EXISTS, false otherwise.
+	 *
+	 * A table may be present in the from list for NOT EXISTS if it is used
+	 * as a correlated NOT EXISTS subquery. In such a situation, when the
+	 * subquery is flattened, it is important that we remember that this is
+	 * a NOT EXISTS subquery, because the join semantics are different 
+	 * (we're looking for rows that do NOT match, rather than rows
+	 * that do). And since the join semantics are different, we cannot
+	 * include this table into a transitive closure of equijoins
+	 * (See DERBY-3033 for a situation where this occurs).
+	 *
+	 * @param tableNumber	which table to check
+	 * @return true if this table is in the from list as NOT EXISTS
+	 */
+	boolean tableNumberIsNotExists(int tableNumber)
+		throws StandardException
+	{
+		int size = size();
+		for (int index = 0; index < size; index++)
+		{
+			ProjectRestrictNode prn = (ProjectRestrictNode) elementAt(index);
+			if (! (prn.getChildResult() instanceof FromTable))
+			{
+				continue;
+			}
+			FromTable ft = (FromTable) prn.getChildResult();
+			if (ft.getTableNumber() == tableNumber)
+				return ft.isNotExists();
+		}
+		return false;
+	}
+	/**
 	 * Get the lock mode for the target of an update statement
 	 * (a delete or update).  The update mode will always be row for
 	 * CurrentOfNodes.  It will be table if there is no where clause.

Modified: db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/PredicateList.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/PredicateList.java?rev=581943&r1=581942&r2=581943&view=diff
==============================================================================
--- db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/PredicateList.java (original)
+++ db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/compile/PredicateList.java Thu Oct  4 08:13:32 2007
@@ -1818,6 +1818,14 @@
      * between outer and inner.  If so, then we simply assign it to the same 
      * equivalence class.  If not, then we add the new equijoin clause.
 	 *
+	 * Note that an equijoin predicate between two tables CANNOT be
+	 * used for transitive closure, if either of the tables is in the
+	 * fromlist for NOT EXISTS. In that case, the join predicate
+	 * actually specifies that the rows from the indicated table must
+	 * NOT exist, and therefore those non-existent rows cannot be
+	 * transitively joined to the other matching tables. See DERBY-3033
+	 * for a description of a situation in which this actually arises.
+	 *
 	 * @param numTables	The number of tables in the query
 	 * @param fromList	The FromList in question.
 	 * @param cc		The CompilerContext to use
@@ -1867,7 +1875,9 @@
 				ColumnReference leftCR = (ColumnReference) left;
 				ColumnReference rightCR = (ColumnReference) right;
 				if (leftCR.getSourceLevel() == rightCR.getSourceLevel() &&
-					leftCR.getTableNumber() != rightCR.getTableNumber())
+					leftCR.getTableNumber() != rightCR.getTableNumber() &&
+					!fromList.tableNumberIsNotExists(leftCR.getTableNumber()) &&
+					!fromList.tableNumberIsNotExists(rightCR.getTableNumber()))
 				{
 					// Add the equijoin clause to each of the lists
 					joinClauses[leftCR.getTableNumber()].addElement(predicate);

Added: db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/ExistsWithSubqueriesTest.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/ExistsWithSubqueriesTest.java?rev=581943&view=auto
==============================================================================
--- db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/ExistsWithSubqueriesTest.java (added)
+++ db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/ExistsWithSubqueriesTest.java Thu Oct  4 08:13:32 2007
@@ -0,0 +1,543 @@
+/*
+ 
+   Derby - Class org.apache.derbyTesting.functionTests.tests.lang.ExistsWithSubqueriesTest
+
+   Licensed to the Apache Software Foundation (ASF) under one or more
+   contributor license agreements.  See the NOTICE file distributed with
+   this work for additional information regarding copyright ownership.
+   The ASF licenses this file to you under the Apache License, Version 2.0
+   (the "License"); you may not use this file except in compliance with
+   the License.  You may obtain a copy of the License at
+ 
+      http://www.apache.org/licenses/LICENSE-2.0
+ 
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.
+ 
+ */
+
+package org.apache.derbyTesting.functionTests.tests.lang;
+
+import junit.framework.*;
+
+import org.apache.derbyTesting.junit.BaseJDBCTestCase;
+import org.apache.derbyTesting.junit.CleanDatabaseTestSetup;
+import org.apache.derbyTesting.junit.JDBC;
+import org.apache.derbyTesting.junit.TestConfiguration;
+
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+import java.sql.PreparedStatement;
+
+/**
+ * This test contains a variety of cases of EXISTS predicates with subqueries.
+ *
+ * Several tests ensure that an EXISTS predicate which wraps a set operation--
+ * meaning a UNION, INTERSECT, or EXCEPT node--returns the correct results.
+ * For example:
+ *
+ *   select * from ( values 'BAD' ) as T
+ *     where exists ((values 1) intersect (values 2))
+ *
+ * should return zero rows. Prompted by DERBY-2370.
+ *
+ * A somewhat unrelated test verifies the DERBY-3033 behavior, which
+ * involves flattening of subqueries with NOT EXISTS predicates. The
+ * issue here is that a flattened NOT EXISTS subquery cannot be used
+ * to perform equi-join transitive closure, because the implied predicate
+ * that results from the flattening is a NOT EQUALS condition.
+ */
+public class ExistsWithSubqueriesTest extends BaseJDBCTestCase {
+    
+    private static final String EXISTS_PREFIX_1 =
+        "select * from ( values 'GOT_A_ROW' ) as T where exists (";
+
+    private static final String EXISTS_PREFIX_2 =
+        "select j from onerow where exists (";
+
+    /**
+     * Create a test with the given name.
+     * @param name name of the test.
+     *
+     */
+    public ExistsWithSubqueriesTest(String name)
+    {
+        super(name);
+    }
+    
+    /**
+     * Return suite with all tests of the class.
+     */
+    public static Test suite()
+    {
+       TestSuite suite = new TestSuite("EXISTS with SET operations");
+
+        /* This is a language/optimization test so behavior will be the
+         * same for embedded and client/server.  Therefore we only need
+         * to run the test against one or the other; we choose embedded.
+         */
+        suite.addTest(
+            TestConfiguration.embeddedSuite(ExistsWithSubqueriesTest.class));
+
+        /* Wrap the suite in a CleanDatabaseTestSetup that will create
+         * and populate the test tables.
+         */
+        return new CleanDatabaseTestSetup(suite) 
+        {
+            /**
+            * Create and populate the test table.
+            */
+            protected void decorateSQL(Statement s) throws SQLException
+            {
+                s.executeUpdate("create table empty (i int)"); 
+                s.executeUpdate("create table onerow (j int)");
+                s.executeUpdate("insert into onerow values 2");
+                s.executeUpdate("create table diffrow (k int)");
+                s.executeUpdate("insert into diffrow values 4");
+                s.executeUpdate("create table tworows (p int)");
+                s.executeUpdate("insert into tworows values 2, 4");
+                s.executeUpdate("create table onerow2col (j1 int, j2 int)");
+                s.executeUpdate("insert into onerow2col values (2, 2)");
+            }
+        };
+    }
+
+    /**
+     * Test queries where the set operation just involves VALUES
+     * expressions.
+     */
+    public void testSetOpsWithVALUES() throws Exception
+    {
+        Statement st = createStatement();
+        String [][] expRS = new String [1][1];
+
+        expRS[0][0] = "GOT_A_ROW";
+        checkQuery(st, expRS, EXISTS_PREFIX_1 + "values 1 union values 1)");
+        checkQuery(st, expRS, EXISTS_PREFIX_1 + "values 1 intersect values 1)");
+        checkQuery(st, expRS, EXISTS_PREFIX_1 + "values 1 except values 0)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 + "values 1 intersect values 0)");
+        checkQuery(st, null, EXISTS_PREFIX_1 + "values 1 except values 1)");
+        st.close();
+    }
+
+    /**
+     * Test queries where the set operation has subqueries which are not
+     * correlated to the outer query.  It's important to check for cases
+     * where we have explicit columns _and_ cases where we have "*" because
+     * the binding codepaths differ and we want to verify both.
+     */
+    public void testNonCorrelatedSetOps() throws Exception
+    {
+        Statement st = createStatement();
+        String [][] expRS = new String [1][1];
+
+        expRS[0][0] = "GOT_A_ROW";
+
+        // Expect 1 row for the following.
+
+        // Simple UNION with "*".
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "select * from diffrow union select * from onerow)");
+
+        // Simple UNION with explicit columns.
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "select k from diffrow union select j from onerow)");
+
+        // Simple INTERSECT with "*".
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "select * from diffrow intersect select 4 from onerow)");
+
+        // Simple INTERSECT with explicit columns.
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "select k from diffrow intersect select 4 from onerow)");
+
+        // Simple EXCEPT with "*".
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "select * from diffrow except select * from onerow)");
+
+        // Simple EXCEPT with explicit columns.
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "select k from diffrow except select j from onerow)");
+
+        // EXCEPT with "*" where left and right children have their
+        // own preds.
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "(select * from tworows where p = 2) except " +
+            "(select * from tworows where p <> 2))");
+
+        // INTERSECT with "*" where left and right children have their
+        // own preds.
+        checkQuery(st, expRS, EXISTS_PREFIX_1 +
+            "(select * from tworows where p = 2) intersect " +
+            "(select * from tworows where p = 2))");
+
+        // Expect 0 rows for the following.  Similar queries to
+        // above except modified to return no rows.
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select i from empty union select * from empty)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select * from onerow intersect select * from empty)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select j from onerow intersect select i from empty)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select * from empty except select * from onerow)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select i from empty except select j from onerow)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select * from onerow intersect select * from diffrow)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select j from onerow intersect select k from diffrow)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select * from onerow except select * from onerow)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "select j from onerow except select j from onerow)");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "(select * from tworows where p = 2) intersect " +
+            "(select * from tworows where p <> 2))");
+
+        checkQuery(st, null, EXISTS_PREFIX_1 +
+            "(select * from tworows where p = 2) except " +
+            "(select * from tworows where p = 2))");
+
+        // Negative cases.  These should fail because "oops" is not
+        // a valid column in ONEROW.
+
+        assertCompileError("42X04", EXISTS_PREFIX_1 +
+            "(select * from onerow where j = 2) intersect " +
+            "(select oops from onerow where j <> 2))");
+
+        assertCompileError("42X04", EXISTS_PREFIX_1 +
+            "(select * from onerow where j = 2) intersect " +
+            "(select * from onerow where oops <> 2))");
+
+        st.close();
+    }
+
+    /**
+     * Test queries where the set operation has subqueries which are
+     * correlated to the outer query.  Subqueries should still be able
+     * reference the outer query table and execute without error.
+     */
+    public void testCorrelatedSetOps() throws Exception
+    {
+        Statement st = createStatement();
+        String [][] expRS = new String [1][1];
+
+        // "2" here is the value that was inserted into "onerow".
+        expRS[0][0] = "2";
+        
+        // Expect 1 row for the following.
+
+        // Right child of UNION has "*" for RCL and references table
+        // from outer query.
+        checkQuery(st, expRS, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 1 union " +
+            "select * from diffrow where onerow.j < k)");
+
+        // Right child of UNION has qualified "*" for RCL and references
+        // table from outer query.
+        checkQuery(st, expRS, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 1 union " +
+            "select diffrow.* from diffrow where onerow.j < k)");
+
+        // Right child of UNION has explicit RCL and references
+        // table from outer query.
+        checkQuery(st, expRS, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 1 union " +
+            "select k from diffrow where onerow.j < k)");
+
+        /* Right child of UNION is itself another EXISTS query whose
+         * child is another set operator (INTERSECT). The INTERSECT in
+         * turn has a right child which references a table from the
+         * outer-most query.
+         */
+        checkQuery(st, expRS, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 0 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from diffrow intersect " +
+            "     select 2 from diffrow where onerow.j < k))");
+
+        /* Right child of UNION is itself another EXISTS query whose
+         * child is another set operator (INTERSECT). The INTERSECT in
+         * turn has a right child which references a table from the
+         * outer-most query.  In this one the INTERSECT returns zero
+         * rows.
+         */
+        checkQuery(st, expRS, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 1 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from empty intersect " +
+            "    select 3 from empty where onerow.j < i))");
+
+        /* Right child of UNION is itself another EXISTS query whose
+         * child is another set operator (INTERSECT). The INTERSECT in
+         * turn has a right child which references 1) a table from the
+         * outer-most query, and 2) a table from the INTERSECT node's
+         * "parent" subquery (i.e. from the UNION's right subquery).
+         */
+        checkQuery(st, expRS, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 1 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from onerow2col intersect " +
+            "    select 3 from empty where onerow.j < diffrow.k))");
+
+        /* Right child of UNION is itself another EXISTS query whose
+         * child is another set operator (INTERSECT). The INTERSECT in
+         * turn has a right child which references 1) a table from the
+         * outer-most query, and 2) a table from the INTERSECT node's
+         * "parent" query.  In addition, add another predicate to the
+         * UNION's right subquery and make that predicate reference
+         * both 1) a table from the outer-most query, and 2) a table
+         * in the subquery's own FROM list.  All of this to ensure
+         * that binding finds the correct columns at all levels of
+         * the query.
+         */
+        checkQuery(st, expRS, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 1 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from onerow2col intersect " +
+            "    select 3 from empty where onerow.j < k) " +
+            "  and (onerow.j < diffrow.k))");
+
+        // Expect 0 rows for the following.  Similar queries to
+        // above except modified to return no rows.
+
+        checkQuery(st, null, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 0 union " +
+            "select * from diffrow where onerow.j > k)");
+
+        checkQuery(st, null, EXISTS_PREFIX_2 +
+              "select 1 from diffrow where 1 = 0 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from diffrow intersect " +
+            "     select 3 from diffrow where onerow.j < k))");
+
+        checkQuery(st, null, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 0 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from empty intersect " +
+            "    select 3 from empty where onerow.j < i))");
+
+        checkQuery(st, null, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 0 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from onerow2col intersect " +
+            "    select 3 from empty where onerow.j < diffrow.k))");
+
+        checkQuery(st, null, EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 0 union " +
+            "select * from diffrow where exists " +
+            "  (select 2 from onerow2col intersect " +
+            "    select 3 from empty where onerow.j < k) " +
+            "  and (onerow.j < diffrow.k))");
+
+        // Negative cases.
+
+        // Should fail because left and right children of the UNION
+        // have different RCL sizes. (NOTE: Would have passed prior
+        // to DERBY-2370, but that was incorrect).
+        assertCompileError("42X58", EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 0 union " +
+            "select * from onerow2col where onerow.j < j)");
+
+        /* Should fail because there is an explicit subquery ("SELECT *")
+         * within the EXISTS query and such a subquery is not allowed to
+         * reference outer tables.  So we will be unable to find the
+         * column "onerow.j" in this case.
+         */
+        assertCompileError("42X04", EXISTS_PREFIX_2 +
+            "select * from (select 1 from diffrow where 1 = 0 " +
+            "union select * from diffrow where onerow.j < k) x)");
+
+        /* Should fail because the UNION's right subquery is trying to
+         * select from an outer table.  While the subquery is allowed
+         * to reference the outer table in expressions, it cannot
+         * include the outer table in its RCL.
+         */
+        assertCompileError("42X10", EXISTS_PREFIX_2 +
+            "select 1 from diffrow where 1 = 1 union " +
+            "select onerow.* from diffrow where onerow.j < k)");
+
+        st.close();
+    }
+
+    /**
+     * Simple helper method to assert the results of the received
+     * query.  If the array representing expected results is null
+     * then we assert that the query returns no rows.
+     */
+    private void checkQuery(Statement st, String [][] expRS,
+        String query) throws Exception
+    {
+        ResultSet rs = st.executeQuery(query);
+        if (expRS == null)
+            JDBC.assertEmpty(rs);
+        else
+            JDBC.assertFullResultSet(rs, expRS);
+        rs.close();
+    }
+
+    /**
+     * Regression test for Derby-3033.
+     *
+     * This method constructs a query with the property that it:
+     * - contains a NOT EXISTS condition against a correlated subquery
+     * - such that if that subquery is flattened, the result is 3 tables
+     *   which all have join predicates on the same key.
+     * The point of the test is that it is *not* correct to construct
+     * a new equijoin predicate between table d3033_a and d3033_c via
+     * transitive closure, because the join condition between d3033_b and
+     * d3033_c is NOT EXISTS.
+     *
+     * In the original bug, the compiler/optimizer erroneously generated
+     * the extra equijoin predicate, which caused NPE exceptions at
+     * runtime due to attempts to reference the non-existent (NOT EXISTS) row.
+     *
+     * So this test succeeds if it gets the right results and no NPE.
+     */
+    public void testDerby3033()
+        throws Exception
+    {
+        setupDerby3033();
+
+        PreparedStatement pstmt = prepareStatement(
+            "select c1, c2_b " +
+            "from (select distinct st.c1,st.c2_b,dsr.c3_a,st.c3_b " +
+            "      from " +
+            "             d3033_a dsr, " +  // Table order matters here!
+            "             d3033_b st " +
+            "      where dsr.c4_a is null " +
+            "      and   dsr.c2 = ? " +
+            "      and   dsr.c1 = st.c1 " +
+            "      and   not exists ( " +
+            "              select 1 " +
+            "              from d3033_c " +
+            "              where d3033_c.c1 = st.c1 " +
+            "              and   d3033_c.c2 = ? " +
+            "              and   d3033_c.c3_c = ? " +
+            "              ) " +
+            ") temp "
+        );
+ 
+        pstmt.setInt(1, 4);
+        pstmt.setInt(2, 4);
+        pstmt.setInt(3, 100);
+ 
+        String [][]expected = {
+            { "1", "100" },
+            { "2", "200" },
+            { "3", "300" },
+        };
+        ResultSet rs = pstmt.executeQuery();
+        JDBC.assertFullResultSet(rs, expected);
+        pstmt.close();
+    }
+
+    /**
+     * Ensure that the #rows statistics are updated
+     */
+    private void updateStats(Statement st, String tName)
+        throws Exception
+    {
+        ResultSet rs = st.executeQuery("select * from " + tName);
+        int numRows = 0;
+        while (rs.next())
+            numRows ++;
+        rs.close();
+    }
+
+    private void setupDerby3033()
+        throws Exception
+    {
+        // The pattern of inserting the data is fairly important, as we
+        // are going to do a combination of joins between the three tables
+        // and we want both matching data and non-matching data. We load:
+        //
+        // d3033_a      d3033_b       d3033_c
+        // --------     --------      --------
+        //    1            1             1
+        //    2            2             3
+        //    3            3
+        //                 4
+        //
+        // We also load a whole pile of irrelevant data into tables a and c
+        // so that the index becomes relevant in the optimizer's analysis,
+        // then we create some constraints and indexes and delete the rows
+        // from table d3033_c (the NOT EXISTS table).
+        //
+        Statement s = createStatement();
+
+        s.executeUpdate("create table d3033_a "+
+                        "(c1 int, c2 int, c3_a int, c4_a date)");
+        s.executeUpdate("create table d3033_b "+
+                        "(c1 int primary key not null, c2_b int, c3_b date)");
+        s.executeUpdate("create table d3033_c (c1 int, c2 int, c3_c int)");
+        s.executeUpdate("insert into d3033_a (c1,c2,c3_a) values(1, 4, 10)");
+        s.executeUpdate("insert into d3033_a (c1,c2,c3_a) values(2, 4, 20)");
+        s.executeUpdate("insert into d3033_a (c1,c2,c3_a) values(3, 4, 30)");
+        s.executeUpdate("insert into d3033_b values(1, 100, CURRENT_DATE)");
+        s.executeUpdate("insert into d3033_b values(2, 200, CURRENT_DATE)");
+        s.executeUpdate("insert into d3033_b values(3, 300, CURRENT_DATE)");
+        s.executeUpdate("insert into d3033_b values(4, 400, CURRENT_DATE)");
+        s.executeUpdate("insert into d3033_c values(1, 4, 100)");
+        s.executeUpdate("insert into d3033_c values(3, 4, 100)");
+            
+        PreparedStatement pstmt2 = prepareStatement(
+                "insert into d3033_a (c1, c2, c3_a) values (?,?,?)");
+
+        PreparedStatement pstmt = prepareStatement(
+                "insert into d3033_b (c1, c2_b, c3_b) values (?,?,?)");
+
+        java.util.Date now = new java.util.Date();
+        java.sql.Timestamp nowTS = new java.sql.Timestamp(now.getTime());
+        for (int i = 0; i < 15; i++)
+        {
+            pstmt.setInt(1, 100+i);
+            pstmt.setInt(2, 100+i);
+            pstmt.setTimestamp(3, nowTS);
+            pstmt.executeUpdate();
+
+            for (int j = 0; j < 200; j++)
+            {
+                pstmt2.setInt(1, 1000+j);
+                pstmt2.setInt(2, 100+i); // note "i" here (FK)
+                pstmt2.setInt(3, 1000 + (j+1)*10);
+                pstmt2.executeUpdate();
+            }
+        }
+
+
+        s.executeUpdate("alter table d3033_a add constraint " +
+                "d3033_a_fk foreign key (c2) references d3033_b(c1) " +
+                "on delete cascade on update no action");
+
+        s.executeUpdate("alter table d3033_c add constraint " +
+                "d3033_c_fk foreign key (c1) references d3033_b(c1) " +
+                "on delete cascade on update no action");
+
+        s.executeUpdate("delete from d3033_c");
+
+        // Update the statistics on the 3 tables:
+        updateStats(s, "d3033_a");
+        updateStats(s, "d3033_b");
+        updateStats(s, "d3033_c");
+
+        s.close();
+    }
+
+}

Propchange: db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/ExistsWithSubqueriesTest.java
------------------------------------------------------------------------------
    svn:eol-style = native

Modified: db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/_Suite.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/_Suite.java?rev=581943&r1=581942&r2=581943&view=diff
==============================================================================
--- db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/_Suite.java (original)
+++ db/derby/code/trunk/java/testing/org/apache/derbyTesting/functionTests/tests/lang/_Suite.java Thu Oct  4 08:13:32 2007
@@ -63,7 +63,7 @@
         suite.addTest(CreateTableFromQueryTest.suite());
         suite.addTest(DatabaseClassLoadingTest.suite());
         suite.addTest(DynamicLikeOptimizationTest.suite());
-        suite.addTest(ExistsWithSetOpsTest.suite());
+        suite.addTest(ExistsWithSubqueriesTest.suite());
         suite.addTest(GrantRevokeTest.suite());
         suite.addTest(GroupByExpressionTest.suite());
 		suite.addTest(LangScripts.suite());