davsclaus commented on PR #27910: URL: https://github.com/apache/camel/pull/27910#issuecomment-6236711658
@etzel-mes, #27906 (CAMEL-25671) is merged now, and it conflicts with this PR in `SubscriptionManager.java`: it removed `badSubscriptions` and the resubscribe that served the old `Bad_InvalidArgument` branch, while `handleSubscriptionError` and `resubscribe()` here still use them. Could you rebase onto main and resolve that against #27906? GitHub does not run the PR checks while the PR conflicts. Two commits on your branch are from the review and should be kept: - 581cdf637118 and 02986bba36e8: the slow route of `MonitorItemLifecycleTest` holds the delivery thread with `.filter(...).delay(3000).syncDelayed().end().end()` instead of `Thread.sleep(3000)` (the delay is a block of its own, hence the two `end()`). - 017e5c7097de is a merge of main for the `ServerCertificateValidationTest` fix (CAMEL-25645); a rebase replaces it. _Claude Code on behalf of Claus Ibsen_ -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
