Executing the statement CREATE INDEX IF ALREADY EXISTS if the index already…

Executing the statement CREATE INDEX IF ALREADY EXISTS if the index already exists no longer fails for a read only database.

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

@@ -58,6 +58,12 @@ public class CreateIndex extends SchemaCommand {
         Database db = session.getDatabase();
         boolean persistent = db.isPersistent();
         Table table = getSchema().getTableOrView(session, tableName);
+        if (getSchema().findIndex(session, indexName) != null) {
+            if (ifNotExists) {
+                return 0;
+            }
+            throw DbException.get(ErrorCode.INDEX_ALREADY_EXISTS_1, indexName);
+        }
         session.getUser().checkRight(table, Right.ALL);
         table.lock(session, true, true);
         if (!table.isPersistIndexes()) {
@@ -71,12 +77,6 @@ public class CreateIndex extends SchemaCommand {
                 indexName = table.getSchema().getUniqueIndexName(session, table, Constants.PREFIX_INDEX);
             }
         }
-        if (getSchema().findIndex(session, indexName) != null) {
-            if (ifNotExists) {
-                return 0;
-            }
-            throw DbException.get(ErrorCode.INDEX_ALREADY_EXISTS_1, indexName);
-        }
         IndexType indexType;
         if (primaryKey) {
             if (table.findPrimaryKey() != null) {

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

@@ -100,9 +100,14 @@ public class TestReadOnly extends TestBase {
     private void testReadOnlyDbCreate() throws SQLException {
         deleteDb("readonly");
         Connection conn = getConnection("readonly");
+        Statement stat = conn.createStatement();
+        stat.execute("create table a(id int)");
+        stat.execute("create index ai on a(id)");
         conn.close();
         conn = getConnection("readonly;ACCESS_MODE_DATA=r");
-        Statement stat = conn.createStatement();
+        stat = conn.createStatement();
+        stat.execute("create table if not exists a(id int)");
+        stat.execute("create index if not exists ai on a(id)");
         assertThrows(ErrorCode.DATABASE_IS_READ_ONLY, stat).
                 execute("CREATE TABLE TEST(ID INT)");
         assertThrows(ErrorCode.TABLE_OR_VIEW_NOT_FOUND_1, stat).