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

yiguolei 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 b70e9498b7d [fix](schema cache) adjust the destruction order of 
_tablet_schema_cache and storage engine (#29923)
b70e9498b7d is described below

commit b70e9498b7d723049ee0b179c9d26817ce873e61
Author: Luwei <[email protected]>
AuthorDate: Sat Jan 13 22:46:07 2024 +0800

    [fix](schema cache) adjust the destruction order of _tablet_schema_cache 
and storage engine (#29923)
---
 be/src/runtime/exec_env_init.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/be/src/runtime/exec_env_init.cpp b/be/src/runtime/exec_env_init.cpp
index 828743ec04d..029d32d9bc6 100644
--- a/be/src/runtime/exec_env_init.cpp
+++ b/be/src/runtime/exec_env_init.cpp
@@ -557,9 +557,6 @@ void ExecEnv::destroy() {
     SAFE_SHUTDOWN(_send_report_thread_pool);
     SAFE_SHUTDOWN(_send_batch_thread_pool);
 
-    // Free resource after threads are stopped.
-    // Some threads are still running, like threads created by 
_new_load_stream_mgr ...
-    SAFE_DELETE(_tablet_schema_cache);
     _deregister_metrics();
     SAFE_DELETE(_load_channel_mgr);
 
@@ -581,6 +578,10 @@ void ExecEnv::destroy() {
     // StorageEngine must be destoried before _cache_manager destory
     SAFE_DELETE(_storage_engine);
 
+    // Free resource after threads are stopped.
+    // Some threads are still running, like threads created by 
_new_load_stream_mgr ...
+    SAFE_DELETE(_tablet_schema_cache);
+
     // _scanner_scheduler must be desotried before _storage_page_cache
     SAFE_DELETE(_scanner_scheduler);
     // _storage_page_cache must be destoried before _cache_manager


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to