[native] Fix exchange HTTP executors shutdown order

prestodb · Jul 24, 2024 · ca11b2f · ca11b2f
1 parent 9d55bda
commit ca11b2f
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/presto-native-execution/presto_cpp/main/PrestoServer.cpp b/presto-native-execution/presto_cpp/main/PrestoServer.cpp
@@ -615,20 +615,20 @@ void PrestoServer::run() {
     httpSrvIOExecutor_->join();
   }
 
-  PRESTO_SHUTDOWN_LOG(INFO)
-      << "Joining Exchange Http IO executor '"
-      << exchangeHttpIoExecutor_->getName()
-      << "': threads: " << exchangeHttpIoExecutor_->numActiveThreads() << "/"
-      << exchangeHttpIoExecutor_->numThreads();
-  exchangeHttpIoExecutor_->join();
-
   PRESTO_SHUTDOWN_LOG(INFO)
       << "Joining Exchange Http CPU executor '"
       << exchangeHttpCpuExecutor_->getName()
       << "': threads: " << exchangeHttpCpuExecutor_->numActiveThreads() << "/"
       << exchangeHttpCpuExecutor_->numThreads();
   exchangeHttpCpuExecutor_->join();
 
+  PRESTO_SHUTDOWN_LOG(INFO)
+      << "Joining Exchange Http IO executor '"
+      << exchangeHttpIoExecutor_->getName()
+      << "': threads: " << exchangeHttpIoExecutor_->numActiveThreads() << "/"
+      << exchangeHttpIoExecutor_->numThreads();
+  exchangeHttpIoExecutor_->join();
+
   PRESTO_SHUTDOWN_LOG(INFO) << "Done joining our executors.";
 
   auto globalCPUKeepAliveExec = folly::getGlobalCPUExecutor();