Issue 517: Create or replace view statement has no effect on the others already…

Issue 517: Create or replace view statement has no effect on the others already existing JDBC connection    

h2/src/docsrc/html/changelog.html

@@ -24,7 +24,8 @@ Change Log
 </li><li>MVStore: concurrency problems have been fixed.
 </li><li>Improve error message when dropping an index that belongs to a constraint,
     specify constraint in error message.
-</li><li>Issue 518: java.sql.Connection.commit() freezes after LOB modification with EXCLUSIVE connection    
+</li><li>Issue 518: java.sql.Connection.commit() freezes after LOB modification with EXCLUSIVE connection
+</li><li>Issue 517: Create or replace view statement has no effect on the others already existing JDBC connection    
 </li></ul>
 
 <h2>Version 1.3.174 (2013-10-19)</h2>

h2/src/main/org/h2/command/ddl/CreateView.java

@@ -107,6 +107,7 @@ public class CreateView extends SchemaCommand {
                 view = new TableView(getSchema(), id, viewName, querySQL, null, columnNames, sysSession, false);
             } else {
                 view.replace(querySQL, columnNames, sysSession, false, force);
+                view.setModified();
             }
         } finally {
             sysSession.setCurrentSchema(db.getSchema(Constants.SCHEMA_MAIN));

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

@@ -42,6 +42,7 @@ public class TestView extends TestBase {
         testUnionReconnect();
         testManyViews();
         testReferenceView();
+        testViewAlterAndCommandCache();
         deleteDb("view");
     }
 
@@ -205,4 +206,25 @@ public class TestView extends TestBase {
         conn.close();
         deleteDb("view");
     }
+    
+    /** make sure that when we change a view, that change in reflected in other sessions command cache */
+    private void testViewAlterAndCommandCache() throws SQLException {
+        deleteDb("view");
+        Connection conn = getConnection("view");
+        Statement stat = conn.createStatement();
+        stat.execute("create table t0(id int primary key)");
+        stat.execute("create table t1(id int primary key)");
+        stat.execute("insert into t0 values(0)");
+        stat.execute("insert into t1 values(1)");
+        stat.execute("create view v1 as select * from t0");
+        ResultSet rs = stat.executeQuery("select * from v1");
+        assertTrue(rs.next());
+        assertEquals(0, rs.getInt(1));
+        stat.execute("create or replace view v1 as select * from t1");
+        rs = stat.executeQuery("select * from v1");
+        assertTrue(rs.next());
+        assertEquals(1, rs.getInt(1));
+        conn.close();
+        deleteDb("view");
+    }
 }