svn commit: r497279 - /incubator/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/NotEqualExpression.java

[mp...@apache.org]

Author: mprudhom
Date: Wed Jan 17 17:30:45 2007
New Revision: 497279

URL: http://svn.apache.org/viewvc?view=rev&rev=497279
Log:
Changed SQL generated by JPQL query "null <> null" from "NULL IS NOT NULL" to "1 <> 1", since some databases (e.g., DB2) dislike "NULL IS NOT NULL", but everyone is OK with "1 <> 1".

Modified:
    incubator/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/NotEqualExpression.java

Modified: incubator/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/NotEqualExpression.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/NotEqualExpression.java?view=diff&rev=497279&r1=497278&r2=497279
==============================================================================
--- incubator/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/NotEqualExpression.java (original)
+++ incubator/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/NotEqualExpression.java Wed Jan 17 17:30:45 2007
@@ -36,7 +36,7 @@
     public void appendTo(Select sel, ExpContext ctx, BinaryOpExpState bstate, 
         SQLBuffer buf, boolean val1Null, boolean val2Null) {
         if (val1Null && val2Null)
-            buf.appendValue(null).append(" IS NOT ").appendValue(null);
+            buf.append("1 <> 1");
         else if (val1Null || val2Null) {
             Val val = (val1Null) ? getValue2() : getValue1();
             ExpState state = (val1Null) ? bstate.state2 : bstate.state1;