[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #1473: Fix #1468 Intermittent failure of CamelDevModeTest

[GitBox <gi...@apache.org>]

lburgazzoli commented on a change in pull request #1473:
URL: https://github.com/apache/camel-quarkus/pull/1473#discussion_r453699833



##########
File path: extensions-core/main/runtime/src/main/java/org/apache/camel/quarkus/main/CamelMain.java
##########
@@ -77,6 +83,9 @@ protected void doStart() throws Exception {
     @Override
     protected void doStop() throws Exception {
         super.doStop();
+        /* Wait till the Camel shutdown is finished in its separate thread and proceed with subsequent Quarkus shutdown
+         * tasks only after that */
+        getShutdownStrategy().await(shutdownTimeoutMs, TimeUnit.MILLISECONDS);

Review comment:
       shouldn't be better to put this in `CamelMainRuntime::stop` metod ? here you are waiting to the shut-down before context.stop() is invoked




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org