Skip to content

Commit 45c1c4f

Browse files
committed
[Bug] BE download_files function exists log print sensitive msg #28592
1 parent 4495f81 commit 45c1c4f

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

be/src/olap/task/engine_clone_task.cpp

+3-2
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,13 @@
3131
#include <memory>
3232
#include <mutex>
3333
#include <ostream>
34+
#include <regex>
3435
#include <set>
3536
#include <shared_mutex>
3637
#include <system_error>
3738
#include <unordered_map>
3839
#include <unordered_set>
3940
#include <utility>
40-
#include <regex>
4141

4242
#include "common/config.h"
4343
#include "common/logging.h"
@@ -547,7 +547,8 @@ Status EngineCloneTask::_download_files(DataDir* data_dir, const std::string& re
547547
}
548548
if (local_file_size != file_size) {
549549
LOG(WARNING) << "download file length error"
550-
<< ", remote_path=" << _mask_token(remote_file_url) << ", file_size=" << file_size
550+
<< ", remote_path=" << _mask_token(remote_file_url)
551+
<< ", file_size=" << file_size
551552
<< ", local_file_size=" << local_file_size;
552553
return Status::InternalError("downloaded file size is not equal");
553554
}

be/src/olap/task/engine_clone_task.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ class EngineCloneTask : public EngineTask {
8585
bool* allow_incremental_clone);
8686

8787
Status _release_snapshot(const std::string& ip, int port, const std::string& snapshot_path);
88-
88+
8989
std::string _mask_token(const std::string& str);
9090

9191
private:

0 commit comments

Comments
 (0)