Merge pull request #711 from pdet/order_deletion

Order deletions since that's required for deleted files

Author: pdet

src/storage/ducklake_metadata_manager.cpp

@@ -2004,7 +2004,8 @@ DuckLakeMetadataManager::ReadAllInlinedDataForFlush(DuckLakeSnapshot snapshot, c
 	auto result = transaction.Query(snapshot, StringUtil::Format(R"(
 SELECT %s
 FROM {METADATA_CATALOG}.%s inlined_data
-WHERE {SNAPSHOT_ID} >= begin_snapshot;)",
+WHERE {SNAPSHOT_ID} >= begin_snapshot
+ORDER BY row_id;)",
 	                                                             projection, inlined_table_name));
 	return TransformInlinedData(*result);
 }

test/configs/postgres.json

@@ -38,14 +38,6 @@
       "paths": [
         "test/sql/transaction/transaction_conflict_inlining.test"
       ]
-    },
-    {
-      "reason": "FIXME: Deletion inlining wrong results?",
-      "paths": [
-        "test/sql/deletion_inlining/test_deletion_multiple_tables.test",
-        "test/sql/deletion_inlining/test_deletion_from_inlined_multiple_snapshots.test",
-        "test/sql/deletion_inlining/test_deletion_from_inlined_insertion.test"
-      ]
     }
   ]
 }