fix optional_idx issue

duckdb · Dec 20, 2024 · 23f4e9c · 23f4e9c
1 parent 1ed221c
commit 23f4e9c
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/storage/delta_catalog.cpp b/src/storage/delta_catalog.cpp
@@ -60,19 +60,24 @@ bool DeltaCatalog::UseCachedSnapshot() {
 
 optional_idx DeltaCatalog::GetCatalogVersion(ClientContext &context) {
 	auto &delta_transaction = DeltaTransaction::Get(context, *this);
+    idx_t version = DConstants::INVALID_INDEX;
 
 	// Option 1: snapshot is cached table-wide
 	auto cached_snapshot = main_schema->GetCachedTable();
 	if (cached_snapshot) {
-		return cached_snapshot->snapshot->GetVersion();
+	    version = cached_snapshot->snapshot->GetVersion();
 	}
 
 	// Option 2: snapshot is cached in transaction
 	if (delta_transaction.table_entry) {
-		return delta_transaction.table_entry->snapshot->GetVersion();
+	    version = delta_transaction.table_entry->snapshot->GetVersion();
 	}
 
-	return {};
+    if (version != DConstants::INVALID_INDEX) {
+        return version;
+    }
+
+	return optional_idx::Invalid();
 }
 
 DatabaseSize DeltaCatalog::GetDatabaseSize(ClientContext &context) {