fix for rollback

core/src/main/java/org/polypheny/db/catalog/impl/PolyCatalog.java

@@ -288,19 +288,14 @@ public synchronized void commit() {
 
 
     public void rollback() {
-        long id = snapshot.id();
-
         commitActions.clear();
         commitConstraints.clear();
 
         restoreLastState();
 
         log.debug( "rollback" );
 
-        if ( id != snapshot.id() ) {
-            updateSnapshot();
-        }
-
+        updateSnapshot();
     }
 
 

dbms/src/test/java/org/polypheny/db/catalog/RollbackTest.java

@@ -0,0 +1,47 @@
+/*
+ * Copyright 2019-2024 The Polypheny Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.polypheny.db.catalog;
+
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.polypheny.db.TestHelper;
+
+@SuppressWarnings("SqlNoDataSourceInspection")
+public class RollbackTest {
+
+    @BeforeAll
+    public static void setUp() {
+        //noinspection ResultOfMethodCallIgnored
+        TestHelper.getInstance();
+    }
+
+    @Test
+    public void failedCreateRollbackTest() {
+        Assertions.assertThrows( Exception.class, () -> {
+            TestHelper.executeSql(
+                    ( c, s ) -> s.executeUpdate( "CREATE TABLE t (a BOOLEAN)" )
+            );
+        } );
+
+        TestHelper.executeSql(
+                ( c, s ) -> s.executeUpdate( "DROP TABLE IF EXISTS t" )
+        );
+
+    }
+
+}