Track max _totalTransactionElapsed across transactions in a command

When a command runs multiple transactions (e.g., peek then process),
_totalTransactionElapsed was only storing the last transaction's time.
This meant slow transactions in earlier phases were silently lost in
timing logs.

Use max() in both rollback and commit so the highest transaction time
is preserved. Reset the counter at command boundaries (start of
peekCommand) so it doesn't carry over between commands on the same
shared db handle.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: danieldoglas

BedrockCore.cpp

@@ -142,6 +142,7 @@ BedrockCore::RESULT BedrockCore::peekCommand(unique_ptr<BedrockCommand>& command
     try {
         SDEBUG("Peeking at '" << request.methodLine << "' with priority: " << command->priority);
         command->peekCount++;
+        _db.resetMaxTransactionElapsed();
         _db.setTimeout(_getRemainingTime(command, false));
         _db.setAbortRef(command->shouldAbort);
 
@@ -240,6 +241,7 @@ BedrockCore::RESULT BedrockCore::processCommand(unique_ptr<BedrockCommand>& comm
     try {
         SDEBUG("Processing '" << request.methodLine << "'");
         command->processCount++;
+        _db.resetMaxTransactionElapsed();
         _db.setTimeout(_getRemainingTime(command, true));
         _db.setAbortRef(command->shouldAbort);
         if (!_db.insideTransaction()) {

sqlitecluster/SQLite.cpp

@@ -936,7 +936,7 @@ int SQLite::commit(const string& description, const string& commandName, functio
 
         _commitElapsed += STimeNow() - before;
         _commitLockElapsed += _sharedData._commitLockTimer.stop();
-        _totalTransactionElapsed = _transactionTimer.stop();
+        _totalTransactionElapsed = max(_totalTransactionElapsed, _transactionTimer.stop());
         _sharedData.incrementCommit(_uncommittedHash);
         _insideTransaction = false;
         _uncommittedHash.clear();
@@ -1019,8 +1019,10 @@ void SQLite::rollback(const string& commandName)
 {
     // Make sure we're actually inside a transaction
     if (_insideTransaction) {
-        // Store the total transaction time only if we were inside one.
-        _totalTransactionElapsed = _transactionTimer.stop();
+        // Store the highest transaction time across all transactions in this command.
+        // We use max() because a single command may run multiple transactions (e.g., peek then process),
+        // and we want to capture the longest one rather than just the last.
+        _totalTransactionElapsed = max(_totalTransactionElapsed, _transactionTimer.stop());
 
         // Cancel this transaction
         if (_autoRolledBack) {

sqlitecluster/SQLite.h

@@ -210,6 +210,13 @@ class SQLite {
     // Cancels the current transaction and rolls it back.
     void rollback(const string& commandName = "");
 
+    // Resets the max transaction elapsed time. Call this at command boundaries to avoid
+    // carrying over timing from a previous command on the same db handle.
+    void resetMaxTransactionElapsed()
+    {
+        _totalTransactionElapsed = 0;
+    }
+
     // Returns the total number of changes on this database
     int getChangeCount()
     {