JinyuChen97 commented on code in PR #25767:
URL: https://github.com/apache/camel/pull/25767#discussion_r3871049725
##########
components/camel-paho-mqtt5/src/main/java/org/apache/camel/component/paho/mqtt5/PahoMqtt5Consumer.java:
##########
@@ -139,6 +141,23 @@ public void deliveryComplete(IMqttToken token) {
}
}
+ private void restartRouteAsync() {
+ Thread restartThread = new Thread(() -> {
+ try {
+ String routeId = getRoute().getRouteId();
+ LOG.info("Stopping route {} for restart after resubscribe
failure", routeId);
+
getEndpoint().getCamelContext().getRouteController().stopRoute(routeId);
Review Comment:
Hi @davsclaus , thanks for the review. The auto-restart now only applies
when the consumer owns the client (stopClient == true). For external provided
clients, it log at error level but do not restart the route, that means the
route will remain in Started state. It's not changing the behavior but do you
think if this is acceptable?
--
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]
