aldettinger commented on pull request #3310: URL: https://github.com/apache/camel-quarkus/pull/3310#issuecomment-974121450
> I would say that the use of DefaultShudownStrategy is preferable in all cases, with one exception - dev mode - because of quick redeploy. (but if in dev-mode camel.main.shutdownTimeout is set, default shutdown strategy will be used) I agree. Maybe I missread that with the current implementation, `DefaultShudownStrategy` is used in DEV mode when camel main is disabled. Do you agree with that ? > As for documentation. Current status was, that NoShutdownStrategy was used in all cases. This fix brings a change, which could affect some users. This is the reason, why I would say that doc should be updated - somewhere. (that default shutdown strategy was changed - mainly for not dev-mode) Ok, it's clearer now. I think James has an idea that this could be in the migration guide. I think it is a good option. -- 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. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
