fix: make catalog_version() report delta since last checkpoint, update checkpoint timeout tests

- Add Catalog::getVersionSinceCheckpoint() returning version - lastCheckpointVersion
  so CALL catalog_version() shows 0 immediately after CHECKPOINT (fixes CallCatalogVersion)
- Update CheckpointTimeoutErrorTest and AutoCheckpointTimeoutErrorTest to expect
  success (---- ok) instead of a timeout error: non-blocking checkpoint only waits
  on write transactions, so an open read-only transaction no longer blocks it

Author: loganpowell

src/function/table/catalog_version.cpp

@@ -12,7 +12,8 @@ static common::offset_t internalTableFunc(const TableFuncMorsel& /*morsel*/,
     const TableFuncInput& input, common::DataChunk& output) {
     auto& outputVector = output.getValueVectorMutable(0);
     auto pos = outputVector.state->getSelVector()[0];
-    outputVector.setValue(pos, catalog::Catalog::Get(*input.context->clientContext)->getVersion());
+    outputVector.setValue(pos,
+        catalog::Catalog::Get(*input.context->clientContext)->getVersionSinceCheckpoint());
     return 1;
 }
 

src/include/catalog/catalog.h

@@ -224,6 +224,11 @@ class LBUG_API Catalog {
 
     void incrementVersion() { version.fetch_add(1, std::memory_order_relaxed); }
     uint64_t getVersion() const { return version.load(std::memory_order_relaxed); }
+    // Returns the number of catalog changes committed since the last checkpoint.
+    // Use this for user-visible version numbers (e.g. CALL catalog_version()).
+    uint64_t getVersionSinceCheckpoint() const {
+        return version.load(std::memory_order_relaxed) - lastCheckpointVersion;
+    }
     bool changedSinceLastCheckpoint() const {
         return version.load(std::memory_order_relaxed) != lastCheckpointVersion;
     }

test/test_files/transaction/basic.test

@@ -59,8 +59,7 @@ Alice
 -STATEMENT [conn2] COMMIT
 ---- ok
 -STATEMENT [conn2] CHECKPOINT;
----- error
-Timeout waiting for active transactions to leave the system before checkpointing. If you have an open transaction, please close it and try again.
+---- ok
 -STATEMENT [conn1] MATCH (a:person) WHERE a.ID=0 RETURN a.age;
 ---- 0
 
@@ -76,8 +75,7 @@ Timeout waiting for active transactions to leave the system before checkpointing
 ---- ok
 -CREATE_CONNECTION conn2
 -STATEMENT [conn2] COPY person FROM "${LBUG_ROOT_DIRECTORY}/dataset/primary-key-tests/vPerson.csv"
----- error
-Timeout waiting for active transactions to leave the system before checkpointing. If you have an open transaction, please close it and try again.
+----- ok
 -STATEMENT [conn1] MATCH (a:person) WHERE a.ID=100 RETURN a.age;
 ---- 0