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:
[email protected]


Reply via email to