You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by fa...@apache.org on 2009/07/14 06:45:15 UTC

svn commit: r793790 - /openjpa/branches/subquery/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java

Author: fancy
Date: Tue Jul 14 04:45:14 2009
New Revision: 793790

URL: http://svn.apache.org/viewvc?rev=793790&view=rev
Log:
cleanup work

Modified:
    openjpa/branches/subquery/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java

Modified: openjpa/branches/subquery/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java
URL: http://svn.apache.org/viewvc/openjpa/branches/subquery/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java?rev=793790&r1=793789&r2=793790&view=diff
==============================================================================
--- openjpa/branches/subquery/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java (original)
+++ openjpa/branches/subquery/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/DBDictionary.java Tue Jul 14 04:45:14 2009
@@ -2222,15 +2222,15 @@
         return fromSQL;
     }
 
-    //if table1 in join is in the main query, table2 is in
-    //subquery, and table2 participates in other joins
-    //in subquery, the join condition can only be placed in 
-    //the where clause in the subquery
     private boolean correlatedJoinCondition(Join join, Select sel) {
         if (!join.isCorrelated())
             return false;
         Iterator itr = sel.getJoinIterator();
         boolean skip = false;
+        // if table1 in join is in the main query, table2 is in
+        // subquery, and table2 participates in other joins
+        // in subquery, the join condition can only be placed in 
+        // the where clause in the subquery.
         while (itr.hasNext()) {
             Join join1 = (Join) itr.next();
             if (join == join1)