Re: [PR] CAMEL-19541: replaced Thread.sleep() in camel-quartz [camel]

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

Re: [PR] CAMEL-19541: replaced Thread.sleep() in camel-quartz [camel]

2023-10-13 Thread via GitHub
orpiske commented on code in PR #11711: URL: https://github.com/apache/camel/pull/11711#discussion_r1358058977 ## components/camel-quartz/src/test/java/org/apache/camel/component/quartz/SpringQuartzConsumerTwoAppsClusteredRecoveryTest.java: ## @@ -55,14 +58,22 @@ public void tes

Re: [PR] CAMEL-19541: replaced Thread.sleep() in camel-quartz [camel]

2023-10-13 Thread via GitHub
oscerd commented on PR #11711: URL: https://github.com/apache/camel/pull/11711#issuecomment-1761100997 This is the one failing: org.apache.camel.routepolicy.quartz.CronScheduledRoutePolicyTest$CronTest8.testScheduledResumeRoutePolicy -- This is an automated message from the Apache Git Ser

Re: [PR] CAMEL-19541: replaced Thread.sleep() in camel-quartz [camel]

2023-10-12 Thread via GitHub
github-actions[bot] commented on PR #11711: URL: https://github.com/apache/camel/pull/11711#issuecomment-1759901512 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committer

[PR] CAMEL-19541: replaced Thread.sleep() in camel-quartz [camel]

2023-10-12 Thread via GitHub
LostArtist opened a new pull request, #11711: URL: https://github.com/apache/camel/pull/11711 Replaced Thread.sleep() in camel-quartz, however CronTest8 in CronScheduledRoutePolicyTest is still unstable in some cases. Sometimes it throws CamelExecutionException even if you catch it. I would