fix a bug

Author: krypdkat

database/db.cpp

@@ -1529,7 +1529,8 @@ bool db_move_log_to_kvrocks(uint16_t epoch, uint64_t logId) {
 }
 
 bool db_move_logs_to_kvrocks_by_range(uint16_t epoch, long long fromLogId, long long toLogId) {
-    if (!g_redis || !g_kvrocks || fromLogId < 0 || toLogId < fromLogId) return false;
+    if (!g_redis || !g_kvrocks || toLogId < fromLogId) return false;
+    if (fromLogId < 0 || toLogId < 0) return true; // nothing to move
 
     try {
         // Build all keys for the range

database/garbageCleaner.cpp

@@ -26,12 +26,15 @@ bool cleanTransactionAndLogsAndSaveToDisk(TickData& td, LogRangesPerTxInTick& lr
     long long min_log_id = INTMAX_MAX;
     long long max_log_id = -1;
     lr.getMinMax(min_log_id, max_log_id);
-    if (!db_move_logs_to_kvrocks_by_range(td.epoch, min_log_id, max_log_id - 1))
+    if (min_log_id != -1 && max_log_id != -1)
     {
-        Logger::get()->error("Failed to add transactions to kvrocks for tick {} - epoch {}", td.tick, td.epoch);
-        return false;
+        if (!db_move_logs_to_kvrocks_by_range(td.epoch, min_log_id, max_log_id - 1))
+        {
+            Logger::get()->error("Failed to move logs to kvrocks for tick {} - epoch {}", td.tick, td.epoch);
+            return false;
+        }
+        db_delete_logs(td.epoch, min_log_id, max_log_id - 1);
     }
-    db_delete_logs(td.epoch, min_log_id, max_log_id - 1);
     return true;
 }
 void compressTickAndMoveToKVRocks(uint32_t tick, FullTickStruct& full, std::vector<TickVote>& votes, std::vector<char>& compressedBuffer)