You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "davsclaus (via GitHub)" <gi...@apache.org> on 2023/02/27 19:52:00 UTC

[GitHub] [camel] davsclaus commented on a diff in pull request #9436: (chores) camel-base-engine: cleanup duplicated code

davsclaus commented on code in PR #9436:
URL: https://github.com/apache/camel/pull/9436#discussion_r1119241272


##########
core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/CamelInternalProcessor.java:
##########
@@ -813,10 +801,10 @@ public void after(Exchange exchange, UnitOfWork uow) throws Exception {
                 UnitOfWorkHelper.doneUow(uow, exchange);
             }
 
-            // after UoW is done lets pop the route context which must be done on every existing UoW
             if (route != null && existing != null) {
                 existing.popRoute();
             }
+            // after UoW is done lets pop the route context which must be done on every existing UoW

Review Comment:
   this comment better belongs where it was before



-- 
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-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org