(kvrocks) branch unstable updated: fix(cmd): always print ok log when get full data file info failed (#2663)

[maplefu]

This is an automated email from the ASF dual-hosted git repository.

maplefu pushed a commit to branch unstable
in repository https://gitbox.apache.org/repos/asf/kvrocks.git


The following commit(s) were added to refs/heads/unstable by this push:
     new e26c54a4 fix(cmd): always print ok log when get full data file info 
failed (#2663)
e26c54a4 is described below

commit e26c54a469e28b627ab44707a5d8aa0a095a17e9
Author: sryan yuan <sr...@qq.com>
AuthorDate: Fri Nov 15 21:50:09 2024 +0800

    fix(cmd): always print ok log when get full data file info failed (#2663)
    
    Co-authored-by: yxj25245 <yxj25...@ly.com>
    Co-authored-by: hulk <hulk.webs...@gmail.com>
---
 src/commands/cmd_replication.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/commands/cmd_replication.cc b/src/commands/cmd_replication.cc
index 4d46e11d..290b1261 100644
--- a/src/commands/cmd_replication.cc
+++ b/src/commands/cmd_replication.cc
@@ -232,11 +232,11 @@ class CommandFetchMeta : public Commander {
       std::string files;
       auto s = 
engine::Storage::ReplDataManager::GetFullReplDataInfo(srv->storage, &files);
       if (!s.IsOK()) {
+        LOG(WARNING) << "[replication] Failed to get full data file info: " << 
s.Msg();
         s = util::SockSend(repl_fd, redis::Error({Status::RedisErrorNoPrefix, 
"can't create db checkpoint"}), bev);
         if (!s.IsOK()) {
           LOG(WARNING) << "[replication] Failed to send error response: " << 
s.Msg();
         }
-        LOG(WARNING) << "[replication] Failed to get full data file info: " << 
s.Msg();
         return;
       }
       // Send full data file info