This is an automated email from the ASF dual-hosted git repository.
zouxinyi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 55d63a8bf37 [opt](memory) Optimize `ClearCacheAction`implementation
(#38438)
55d63a8bf37 is described below
commit 55d63a8bf37775ba2a2d9dba123b9fc69eab58fd
Author: Xinyi Zou <zouxiny...@gmail.com>
AuthorDate: Fri Aug 2 10:01:07 2024 +0800
[opt](memory) Optimize `ClearCacheAction`implementation (#38438)
1. `ClearCacheAction` support for specifying cache,
`http://beip:8040/api/clear_cache/all`,
`http://beip:8040/api/clear_cache/DataPageCache`
2. Optimize result returned by `ClearCacheAction`.
3. Optimize Allocator large memory printing stacktrace
4. `cache_prune_stale_thread` no longer check conf to prune cache, which
has delays and brings a bad experience.
---
be/src/common/config.cpp | 4 +--
be/src/common/config.h | 12 +++-----
be/src/common/daemon.cpp | 23 ++++++++++++++
be/src/common/daemon.h | 1 +
be/src/http/action/clear_cache_action.cpp | 33 ++++++++++++++++++--
be/src/http/action/clear_cache_action.h | 6 ++--
be/src/olap/olap_server.cpp | 42 --------------------------
be/src/olap/storage_engine.h | 6 ----
be/src/runtime/memory/cache_manager.cpp | 20 ++++++------
be/src/runtime/memory/cache_manager.h | 7 ++---
be/src/runtime/memory/cache_policy.h | 29 ++++++++++++++++++
be/src/runtime/memory/memory_reclamation.cpp | 2 --
be/src/runtime/memory/thread_mem_tracker_mgr.h | 4 +--
be/src/service/http_service.cpp | 8 ++---
be/src/vec/common/allocator.cpp | 36 ++++++++++++----------
15 files changed, 130 insertions(+), 103 deletions(-)
diff --git a/be/src/common/config.cpp b/be/src/common/config.cpp
index d0a2a5fa7e3..a68b4c60c01 100644
--- a/be/src/common/config.cpp
+++ b/be/src/common/config.cpp
@@ -137,9 +137,7 @@ DEFINE_mBool(disable_memory_gc, "false");
DEFINE_mBool(enable_stacktrace, "true");
-DEFINE_mBool(enable_stacktrace_in_allocator_check_failed, "false");
-
-DEFINE_mInt64(large_memory_check_bytes, "2147483648");
+DEFINE_mInt64(stacktrace_in_alloc_large_memory_bytes, "2147483648");
DEFINE_mBool(enable_memory_orphan_check, "false");
diff --git a/be/src/common/config.h b/be/src/common/config.h
index e117c824329..3c43ed66593 100644
--- a/be/src/common/config.h
+++ b/be/src/common/config.h
@@ -186,13 +186,11 @@ DECLARE_mBool(disable_memory_gc);
// if false, turn off all stacktrace
DECLARE_mBool(enable_stacktrace);
-// Allocator check failed log stacktrace if not catch exception
-DECLARE_mBool(enable_stacktrace_in_allocator_check_failed);
-
-// malloc or new large memory larger than large_memory_check_bytes, default 2G,
-// will print a warning containing the stacktrace, but not prevent memory
alloc.
-// If is -1, disable large memory check.
-DECLARE_mInt64(large_memory_check_bytes);
+// when alloc memory larger than stacktrace_in_alloc_large_memory_bytes,
default 2G,
+// if alloc successful, will print a warning with stacktrace, but not prevent
memory alloc.
+// if alloc failed using Doris Allocator, will print stacktrace in error log.
+// if is -1, disable print stacktrace when alloc large memory.
+DECLARE_mInt64(stacktrace_in_alloc_large_memory_bytes);
// default is true. if any memory tracking in Orphan mem tracker will report
error.
DECLARE_mBool(enable_memory_orphan_check);
diff --git a/be/src/common/daemon.cpp b/be/src/common/daemon.cpp
index bef9b8798f6..d8245f4045c 100644
--- a/be/src/common/daemon.cpp
+++ b/be/src/common/daemon.cpp
@@ -388,11 +388,29 @@ void Daemon::je_purge_dirty_pages_thread() const {
if (_stop_background_threads_latch.count() == 0) {
break;
}
+ if (config::disable_memory_gc) {
+ continue;
+ }
doris::MemInfo::je_purge_all_arena_dirty_pages();
doris::MemInfo::je_purge_dirty_pages_notify.store(false,
std::memory_order_relaxed);
} while (true);
}
+void Daemon::cache_prune_stale_thread() {
+ int32_t interval = config::cache_periodic_prune_stale_sweep_sec;
+ while
(!_stop_background_threads_latch.wait_for(std::chrono::seconds(interval))) {
+ if (interval <= 0) {
+ LOG(WARNING) << "config of cache clean interval is illegal: [" <<
interval
+ << "], force set to 3600 ";
+ interval = 3600;
+ }
+ if (config::disable_memory_gc) {
+ continue;
+ }
+ CacheManager::instance()->for_each_cache_prune_stale();
+ }
+}
+
void Daemon::wg_weighted_memory_ratio_refresh_thread() {
// Refresh weighted memory ratio of workload groups
while (!_stop_background_threads_latch.wait_for(
@@ -436,6 +454,11 @@ void Daemon::start() {
st = Thread::create(
"Daemon", "je_purge_dirty_pages_thread",
[this]() { this->je_purge_dirty_pages_thread(); },
&_threads.emplace_back());
+ CHECK(st.ok()) << st;
+ st = Thread::create(
+ "Daemon", "cache_prune_stale_thread", [this]() {
this->cache_prune_stale_thread(); },
+ &_threads.emplace_back());
+ CHECK(st.ok()) << st;
st = Thread::create(
"Daemon", "query_runtime_statistics_thread",
[this]() { this->report_runtime_query_statistics_thread(); },
&_threads.emplace_back());
diff --git a/be/src/common/daemon.h b/be/src/common/daemon.h
index 2a8adf20e46..64c9f0c8993 100644
--- a/be/src/common/daemon.h
+++ b/be/src/common/daemon.h
@@ -43,6 +43,7 @@ private:
void memtable_memory_refresh_thread();
void calculate_metrics_thread();
void je_purge_dirty_pages_thread() const;
+ void cache_prune_stale_thread();
void report_runtime_query_statistics_thread();
void wg_weighted_memory_ratio_refresh_thread();
void be_proc_monitor_thread();
diff --git a/be/src/http/action/clear_cache_action.cpp
b/be/src/http/action/clear_cache_action.cpp
index f42499090c4..cb183a99cf1 100644
--- a/be/src/http/action/clear_cache_action.cpp
+++ b/be/src/http/action/clear_cache_action.cpp
@@ -30,10 +30,37 @@ namespace doris {
const static std::string HEADER_JSON = "application/json";
-void ClearDataCacheAction::handle(HttpRequest* req) {
+void ClearCacheAction::handle(HttpRequest* req) {
req->add_output_header(HttpHeaders::CONTENT_TYPE, "text/plain;
version=0.0.4");
- CacheManager::instance()->clear_once();
- HttpChannel::send_reply(req, HttpStatus::OK, "");
+ std::string cache_type_str = req->param("type");
+ fmt::memory_buffer return_string_buffer;
+ int64_t freed_size = 0;
+ if (cache_type_str == "all") {
+ freed_size =
CacheManager::instance()->for_each_cache_prune_all(nullptr, true);
+ } else {
+ CachePolicy::CacheType cache_type =
CachePolicy::string_to_type(cache_type_str);
+ if (cache_type == CachePolicy::CacheType::NONE) {
+ fmt::format_to(return_string_buffer,
+ "ClearCacheAction not match type:{} of cache
policy", cache_type_str);
+ LOG(WARNING) << fmt::to_string(return_string_buffer);
+ HttpChannel::send_reply(req, HttpStatus::INTERNAL_SERVER_ERROR,
+ fmt::to_string(return_string_buffer));
+ return;
+ }
+ freed_size = CacheManager::instance()->cache_prune_all(cache_type,
true);
+ if (freed_size == -1) {
+ fmt::format_to(return_string_buffer,
+ "ClearCacheAction cache:{} is not allowed to be
pruned", cache_type_str);
+ LOG(WARNING) << fmt::to_string(return_string_buffer);
+ HttpChannel::send_reply(req, HttpStatus::INTERNAL_SERVER_ERROR,
+ fmt::to_string(return_string_buffer));
+ return;
+ }
+ }
+ fmt::format_to(return_string_buffer, "ClearCacheAction cache:{} prune win,
freed size {}",
+ cache_type_str, freed_size);
+ LOG(WARNING) << fmt::to_string(return_string_buffer);
+ HttpChannel::send_reply(req, HttpStatus::OK,
fmt::to_string(return_string_buffer));
}
} // end namespace doris
diff --git a/be/src/http/action/clear_cache_action.h
b/be/src/http/action/clear_cache_action.h
index 3840f63593f..3795a87b5d7 100644
--- a/be/src/http/action/clear_cache_action.h
+++ b/be/src/http/action/clear_cache_action.h
@@ -23,11 +23,11 @@ namespace doris {
class HttpRequest;
-class ClearDataCacheAction : public HttpHandler {
+class ClearCacheAction : public HttpHandler {
public:
- ClearDataCacheAction() = default;
+ ClearCacheAction() = default;
- ~ClearDataCacheAction() override = default;
+ ~ClearCacheAction() override = default;
void handle(HttpRequest* req) override;
};
diff --git a/be/src/olap/olap_server.cpp b/be/src/olap/olap_server.cpp
index 345c1bed4ff..7c88156f74c 100644
--- a/be/src/olap/olap_server.cpp
+++ b/be/src/olap/olap_server.cpp
@@ -301,12 +301,6 @@ Status StorageEngine::start_bg_threads() {
[this]() { this->_tablet_path_check_callback(); },
&_tablet_path_check_thread));
LOG(INFO) << "tablet path check thread started";
- // cache clean thread
- RETURN_IF_ERROR(Thread::create(
- "StorageEngine", "cache_clean_thread", [this]() {
this->_cache_clean_callback(); },
- &_cache_clean_thread));
- LOG(INFO) << "cache clean thread started";
-
// path scan and gc thread
if (config::path_gc_check) {
for (auto data_dir : get_stores()) {
@@ -359,42 +353,6 @@ Status StorageEngine::start_bg_threads() {
return Status::OK();
}
-void StorageEngine::_cache_clean_callback() {
- int32_t interval = config::cache_periodic_prune_stale_sweep_sec;
- while
(!_stop_background_threads_latch.wait_for(std::chrono::seconds(interval))) {
- if (interval <= 0) {
- LOG(WARNING) << "config of cache clean interval is illegal: [" <<
interval
- << "], force set to 3600 ";
- interval = 3600;
- }
- if (config::disable_memory_gc) {
- continue;
- }
-
- CacheManager::instance()->for_each_cache_prune_stale();
-
- // Dynamically modify the config to clear the cache, each time the
disable cache will only be cleared once.
- if (config::disable_segment_cache) {
- if (!_clear_segment_cache) {
-
CacheManager::instance()->clear_once(CachePolicy::CacheType::SEGMENT_CACHE);
- _clear_segment_cache = true;
- }
- } else {
- _clear_segment_cache = false;
- }
- if (config::disable_storage_page_cache) {
- if (!_clear_page_cache) {
-
CacheManager::instance()->clear_once(CachePolicy::CacheType::DATA_PAGE_CACHE);
-
CacheManager::instance()->clear_once(CachePolicy::CacheType::INDEXPAGE_CACHE);
-
CacheManager::instance()->clear_once(CachePolicy::CacheType::PK_INDEX_PAGE_CACHE);
- _clear_page_cache = true;
- }
- } else {
- _clear_page_cache = false;
- }
- }
-}
-
void StorageEngine::_garbage_sweeper_thread_callback() {
uint32_t max_interval = config::max_garbage_sweep_interval;
uint32_t min_interval = config::min_garbage_sweep_interval;
diff --git a/be/src/olap/storage_engine.h b/be/src/olap/storage_engine.h
index 3105c4f53d0..7eb94047756 100644
--- a/be/src/olap/storage_engine.h
+++ b/be/src/olap/storage_engine.h
@@ -361,9 +361,6 @@ private:
// delete tablet with io error process function
void _disk_stat_monitor_thread_callback();
- // clean file descriptors cache
- void _cache_clean_callback();
-
// path gc process function
void _path_gc_thread_callback(DataDir* data_dir);
@@ -528,9 +525,6 @@ private:
scoped_refptr<Thread> _async_publish_thread;
std::shared_mutex _async_publish_lock;
- bool _clear_segment_cache = false;
- bool _clear_page_cache = false;
-
std::atomic<bool> _need_clean_trash {false};
// next index for create tablet
diff --git a/be/src/runtime/memory/cache_manager.cpp
b/be/src/runtime/memory/cache_manager.cpp
index 9bf3d1e12d0..a6516c40a35 100644
--- a/be/src/runtime/memory/cache_manager.cpp
+++ b/be/src/runtime/memory/cache_manager.cpp
@@ -48,24 +48,22 @@ int64_t
CacheManager::for_each_cache_prune_stale(RuntimeProfile* profile) {
return 0;
}
-int64_t CacheManager::for_each_cache_prune_all(RuntimeProfile* profile) {
- if (need_prune(&_last_prune_all_timestamp, "all")) {
+int64_t CacheManager::for_each_cache_prune_all(RuntimeProfile* profile, bool
force) {
+ if (force || need_prune(&_last_prune_all_timestamp, "all")) {
return for_each_cache_prune_stale_wrap(
- [](CachePolicy* cache_policy) {
cache_policy->prune_all(false); }, profile);
+ [force](CachePolicy* cache_policy) {
cache_policy->prune_all(force); }, profile);
}
return 0;
}
-void CacheManager::clear_once() {
+int64_t CacheManager::cache_prune_all(CachePolicy::CacheType type, bool force)
{
std::lock_guard<std::mutex> l(_caches_lock);
- for (const auto& pair : _caches) {
- pair.second->prune_all(true);
+ auto* cache_policy = _caches[type];
+ if (!cache_policy->enable_prune()) {
+ return -1;
}
-}
-
-void CacheManager::clear_once(CachePolicy::CacheType type) {
- std::lock_guard<std::mutex> l(_caches_lock);
- _caches[type]->prune_all(true); // will print log
+ cache_policy->prune_all(force);
+ return cache_policy->profile()->get_counter("FreedMemory")->value();
}
} // namespace doris
diff --git a/be/src/runtime/memory/cache_manager.h
b/be/src/runtime/memory/cache_manager.h
index 20372366aa1..d94dca50167 100644
--- a/be/src/runtime/memory/cache_manager.h
+++ b/be/src/runtime/memory/cache_manager.h
@@ -64,10 +64,9 @@ public:
int64_t for_each_cache_prune_stale(RuntimeProfile* profile = nullptr);
- int64_t for_each_cache_prune_all(RuntimeProfile* profile = nullptr);
-
- void clear_once();
- void clear_once(CachePolicy::CacheType type);
+ // if force is true, regardless of the two prune interval and cache size,
cache will be pruned this time.
+ int64_t for_each_cache_prune_all(RuntimeProfile* profile = nullptr, bool
force = false);
+ int64_t cache_prune_all(CachePolicy::CacheType type, bool force = false);
bool need_prune(int64_t* last_timestamp, const std::string& type) {
int64_t now = UnixSeconds();
diff --git a/be/src/runtime/memory/cache_policy.h
b/be/src/runtime/memory/cache_policy.h
index e59c5c7ac3e..c457afd8689 100644
--- a/be/src/runtime/memory/cache_policy.h
+++ b/be/src/runtime/memory/cache_policy.h
@@ -47,6 +47,7 @@ public:
CREATE_TABLET_RR_IDX_CACHE = 15,
CLOUD_TABLET_CACHE = 16,
CLOUD_TXN_DELETE_BITMAP_CACHE = 17,
+ NONE = 18, // not be used
};
static std::string type_string(CacheType type) {
@@ -94,6 +95,34 @@ public:
__builtin_unreachable();
}
+ inline static std::unordered_map<std::string, CacheType> StringToType = {
+ {"DataPageCache", CacheType::DATA_PAGE_CACHE},
+ {"IndexPageCache", CacheType::INDEXPAGE_CACHE},
+ {"PKIndexPageCache", CacheType::PK_INDEX_PAGE_CACHE},
+ {"SchemaCache", CacheType::SCHEMA_CACHE},
+ {"SegmentCache", CacheType::SEGMENT_CACHE},
+ {"InvertedIndexSearcherCache",
CacheType::INVERTEDINDEX_SEARCHER_CACHE},
+ {"InvertedIndexQueryCache", CacheType::INVERTEDINDEX_QUERY_CACHE},
+ {"PointQueryLookupConnectionCache",
CacheType::LOOKUP_CONNECTION_CACHE},
+ {"PointQueryRowCache", CacheType::POINT_QUERY_ROW_CACHE},
+ {"MowDeleteBitmapAggCache", CacheType::DELETE_BITMAP_AGG_CACHE},
+ {"MowTabletVersionCache", CacheType::TABLET_VERSION_CACHE},
+ {"LastSuccessChannelCache", CacheType::LAST_SUCCESS_CHANNEL_CACHE},
+ {"CommonObjLRUCache", CacheType::COMMON_OBJ_LRU_CACHE},
+ {"ForUT", CacheType::FOR_UT},
+ {"TabletSchemaCache", CacheType::TABLET_SCHEMA_CACHE},
+ {"CreateTabletRRIdxCache", CacheType::CREATE_TABLET_RR_IDX_CACHE},
+ {"CloudTabletCache", CacheType::CLOUD_TABLET_CACHE},
+ {"CloudTxnDeleteBitmapCache",
CacheType::CLOUD_TXN_DELETE_BITMAP_CACHE}};
+
+ static CacheType string_to_type(std::string type) {
+ if (StringToType.contains(type)) {
+ return StringToType[type];
+ } else {
+ return CacheType::NONE;
+ }
+ }
+
CachePolicy(CacheType type, uint32_t stale_sweep_time_s, bool
enable_prune);
virtual ~CachePolicy();
diff --git a/be/src/runtime/memory/memory_reclamation.cpp
b/be/src/runtime/memory/memory_reclamation.cpp
index 536c4658c8c..3adf1d1ac75 100644
--- a/be/src/runtime/memory/memory_reclamation.cpp
+++ b/be/src/runtime/memory/memory_reclamation.cpp
@@ -47,7 +47,6 @@ bool MemoryReclamation::process_minor_gc(std::string
mem_info) {
}};
freed_mem +=
CacheManager::instance()->for_each_cache_prune_stale(profile.get());
- MemInfo::notify_je_purge_dirty_pages();
if (freed_mem > MemInfo::process_minor_gc_size()) {
return true;
}
@@ -98,7 +97,6 @@ bool MemoryReclamation::process_full_gc(std::string mem_info)
{
}};
freed_mem +=
CacheManager::instance()->for_each_cache_prune_all(profile.get());
- MemInfo::notify_je_purge_dirty_pages();
if (freed_mem > MemInfo::process_full_gc_size()) {
return true;
}
diff --git a/be/src/runtime/memory/thread_mem_tracker_mgr.h
b/be/src/runtime/memory/thread_mem_tracker_mgr.h
index 141c54382cf..d9c4e093a4a 100644
--- a/be/src/runtime/memory/thread_mem_tracker_mgr.h
+++ b/be/src/runtime/memory/thread_mem_tracker_mgr.h
@@ -242,8 +242,8 @@ inline void ThreadMemTrackerMgr::consume(int64_t size, int
skip_large_memory_che
flush_untracked_mem();
}
- if (skip_large_memory_check == 0 &&
doris::config::large_memory_check_bytes > 0 &&
- size > doris::config::large_memory_check_bytes) {
+ if (skip_large_memory_check == 0 &&
doris::config::stacktrace_in_alloc_large_memory_bytes > 0 &&
+ size > doris::config::stacktrace_in_alloc_large_memory_bytes) {
_stop_consume = true;
LOG(WARNING) << fmt::format(
"malloc or new large memory: {}, {}, this is just a warning,
not prevent memory "
diff --git a/be/src/service/http_service.cpp b/be/src/service/http_service.cpp
index b851302aaa8..9522f23e3bd 100644
--- a/be/src/service/http_service.cpp
+++ b/be/src/service/http_service.cpp
@@ -156,10 +156,10 @@ Status HttpService::start() {
HealthAction* health_action = _pool.add(new HealthAction());
_ev_http_server->register_handler(HttpMethod::GET, "/api/health",
health_action);
- // Dump all running pipeline tasks
- ClearDataCacheAction* clear_data_cache_action = _pool.add(new
ClearDataCacheAction());
- _ev_http_server->register_handler(HttpMethod::GET, "/api/clear_data_cache",
- clear_data_cache_action);
+ // Clear cache action
+ ClearCacheAction* clear_cache_action = _pool.add(new ClearCacheAction());
+ _ev_http_server->register_handler(HttpMethod::GET,
"/api/clear_cache/{type}",
+ clear_cache_action);
// Dump all running pipeline tasks
PipelineTaskAction* pipeline_task_action = _pool.add(new
PipelineTaskAction());
diff --git a/be/src/vec/common/allocator.cpp b/be/src/vec/common/allocator.cpp
index 879e98d0ca4..ed25b6a8f77 100644
--- a/be/src/vec/common/allocator.cpp
+++ b/be/src/vec/common/allocator.cpp
@@ -93,9 +93,8 @@ void Allocator<clear_memory_, mmap_populate, use_mmap,
MemoryAllocator>::sys_mem
doris::thread_context()->thread_mem_tracker_mgr->last_consumer_tracker(),
doris::GlobalMemoryArbitrator::process_limit_exceeded_errmsg_str());
- if (!doris::enable_thread_catch_bad_alloc &&
- (size > 1024L * 1024 * 1024 ||
- doris::config::enable_stacktrace_in_allocator_check_failed)) {
+ if (doris::config::stacktrace_in_alloc_large_memory_bytes > 0 &&
+ size > doris::config::stacktrace_in_alloc_large_memory_bytes) {
err_msg += "\nAlloc Stacktrace:\n" + doris::get_stack_trace();
}
@@ -106,8 +105,11 @@ void Allocator<clear_memory_, mmap_populate, use_mmap,
MemoryAllocator>::sys_mem
}
return;
}
- if (!doris::config::disable_memory_gc &&
- doris::thread_context()->thread_mem_tracker_mgr->is_attach_query()
&&
+
+ // no significant impact on performance is expected.
+ doris::MemInfo::notify_je_purge_dirty_pages();
+
+ if (doris::thread_context()->thread_mem_tracker_mgr->is_attach_query()
&&
doris::thread_context()->thread_mem_tracker_mgr->wait_gc()) {
int64_t wait_milliseconds = 0;
LOG(INFO) << fmt::format(
@@ -115,19 +117,21 @@ void Allocator<clear_memory_, mmap_populate, use_mmap,
MemoryAllocator>::sys_mem
print_id(doris::thread_context()->task_id()),
doris::thread_context()->get_thread_id(),
doris::config::thread_wait_gc_max_milliseconds, err_msg);
- while (wait_milliseconds <
doris::config::thread_wait_gc_max_milliseconds) {
- std::this_thread::sleep_for(std::chrono::milliseconds(100));
- if
(!doris::GlobalMemoryArbitrator::is_exceed_hard_mem_limit(size)) {
-
doris::GlobalMemoryArbitrator::refresh_interval_memory_growth += size;
- break;
- }
- if
(doris::thread_context()->thread_mem_tracker_mgr->is_query_cancelled()) {
- if (doris::enable_thread_catch_bad_alloc) {
- throw
doris::Exception(doris::ErrorCode::MEM_ALLOC_FAILED, err_msg);
+ if (!doris::config::disable_memory_gc) {
+ while (wait_milliseconds <
doris::config::thread_wait_gc_max_milliseconds) {
+
std::this_thread::sleep_for(std::chrono::milliseconds(100));
+ if
(!doris::GlobalMemoryArbitrator::is_exceed_hard_mem_limit(size)) {
+
doris::GlobalMemoryArbitrator::refresh_interval_memory_growth += size;
+ break;
+ }
+ if
(doris::thread_context()->thread_mem_tracker_mgr->is_query_cancelled()) {
+ if (doris::enable_thread_catch_bad_alloc) {
+ throw
doris::Exception(doris::ErrorCode::MEM_ALLOC_FAILED, err_msg);
+ }
+ return;
}
- return;
+ wait_milliseconds += 100;
}
- wait_milliseconds += 100;
}
if (wait_milliseconds >=
doris::config::thread_wait_gc_max_milliseconds) {
// Make sure to completely wait
thread_wait_gc_max_milliseconds only once.
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
