Re: [PR] CAMEL-19526: replaced the Thread.sleep with the best option or removed it. [camel]

2023-11-10 Thread via GitHub
orpiske merged PR #11945: URL: https://github.com/apache/camel/pull/11945 -- 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:

Re: [PR] CAMEL-19526: replaced the Thread.sleep with the best option or removed it. [camel]

2023-11-09 Thread via GitHub
rafamqrs commented on PR #11945: URL: https://github.com/apache/camel/pull/11945#issuecomment-1804769570 > Thanks for your contribution! > > There are few places where I think it would be better to retain the `Thread.sleep` because the proposed solution with Awaitility makes the code

Re: [PR] CAMEL-19526: replaced the Thread.sleep with the best option or removed it. [camel]

2023-11-09 Thread via GitHub
rafamqrs commented on code in PR #11945: URL: https://github.com/apache/camel/pull/11945#discussion_r1388640552 ## components/camel-jms/src/test/java/org/apache/camel/component/jms/temp/JmsReconnectManualTest.java: ## @@ -102,7 +105,9 @@ public void configure() { *