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 8f7e7a7b31 [Fix](signal) fix signal handler (#24144)
8f7e7a7b31 is described below

commit 8f7e7a7b319727a6a9129ad131ed801132e91004
Author: zhiqqqq <seuhezhiqi...@163.com>
AuthorDate: Mon Sep 11 13:18:49 2023 +0800

    [Fix](signal) fix signal handler (#24144)
---
 be/src/service/doris_main.cpp | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/be/src/service/doris_main.cpp b/be/src/service/doris_main.cpp
index 2370cd3f95..fd55c5a211 100644
--- a/be/src/service/doris_main.cpp
+++ b/be/src/service/doris_main.cpp
@@ -308,7 +308,6 @@ struct Checker {
 
 int main(int argc, char** argv) {
     doris::signal::InstallFailureSignalHandler();
-    doris::init_signals();
     // create StackTraceCache Instance, at the beginning, other static 
destructors may use.
     StackTrace::createCache();
 
@@ -441,9 +440,16 @@ int main(int argc, char** argv) {
         if (!status.ok()) {
             LOG(WARNING) << "Failed to initialize JNI: " << status;
             exit(1);
+        } else {
+            LOG(INFO) << "Doris backend JNI is initialized.";
         }
     }
 
+    // Doris own signal handler must be register after jvm is init.
+    // Or our own sig-handler for SIGINT & SIGTERM will not be chained ...
+    // https://www.oracle.com/java/technologies/javase/signals.html
+    doris::init_signals();
+
     // Load file cache before starting up daemon threads to make sure 
StorageEngine is read.
     if (doris::config::enable_file_cache) {
         doris::io::IFileCache::init();


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to