In version 1.3.183, indexes that contains columns with a default value generated…

In version 1.3.183, indexes that contains columns with a default value generated by a sequence where not used. This includes identity and auto-increment columns. This bug was introduced by supporting "rownum" in views and derived tables.

h2/src/docsrc/html/changelog.html

@@ -17,7 +17,9 @@ Change Log
 <h1>Change Log</h1>
 
 <h2>Next Version (unreleased)</h2>
-<ul><li>-
+<ul><li>In version 1.3.183, indexes that contains columns with a default value 
+    generated by a sequence where not used. This includes identity and auto-increment 
+    columns. This bug was introduced by supporting "rownum" in views and derived tables.
 </li></ul>
 
 <h2>Version 1.4.183 Beta (2014-12-13)</h2>

h2/src/main/org/h2/table/Table.java

@@ -333,12 +333,6 @@ public abstract class Table extends SchemaObjectBase {
      * @return true if the table contains no query-comparable column
      */
     public boolean isQueryComparable() {
-        ExpressionVisitor visitor = ExpressionVisitor.QUERY_COMPARABLE_VISITOR;
-        for (Column col : columns) {
-            if (!col.isEverything(visitor)) {
-                return false;
-            }
-        }
         return true;
     }
 

h2/src/test/org/h2/test/db/TestOptimizations.java

@@ -41,6 +41,7 @@ public class TestOptimizations extends TestBase {
     @Override
     public void test() throws Exception {
         deleteDb("optimizations");
+        testIdentityIndexUsage();
         testFastRowIdCondition();
         testExplainRoundTrip();
         testOrderByExpression();
@@ -80,6 +81,18 @@ public class TestOptimizations extends TestBase {
         deleteDb("optimizations");
     }
     
+    private void testIdentityIndexUsage() throws Exception {
+        Connection conn = getConnection("optimizations");
+        Statement stat = conn.createStatement();
+        stat.execute("create table test(a identity)");
+        stat.execute("insert into test values()");
+        ResultSet rs = stat.executeQuery("explain select * from test where a = 1");
+        rs.next();
+        assertContains(rs.getString(1), "PRIMARY_KEY");
+        stat.execute("drop table test");
+        conn.close();
+    }
+
     private void testFastRowIdCondition() throws Exception {
         Connection conn = getConnection("optimizations");
         Statement stat = conn.createStatement();