fix: Pika can not exec full-sync when multi slaves connect to the sam…

…e master within a short time (OpenAtomFoundation#2746) * use int64_t instead of int32_t --------- Co-authored-by: cjh <[email protected]>
cheniujh · Jun 24, 2024 · 56b3f95 · 56b3f95
1 parent 8c9b4d0
commit 56b3f95
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/pika_server.cc b/src/pika_server.cc
@@ -833,7 +833,8 @@ void PikaServer::TryDBSync(const std::string& ip, int port, const std::string& d
   std::string logger_filename = sync_db->Logger()->filename();
   if (pstd::IsDir(bgsave_info.path) != 0 ||
       !pstd::FileExists(NewFileName(logger_filename, bgsave_info.offset.b_offset.filenum)) ||
-      top - bgsave_info.offset.b_offset.filenum > kDBSyncMaxGap) {
+      static_cast<int64_t>(top) - static_cast<int64_t>(bgsave_info.offset.b_offset.filenum) >
+          static_cast<int64_t>(kDBSyncMaxGap)) {
     // Need Bgsave first
     db->BgSaveDB();
   }