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


##########
components/camel-seda/src/main/java/org/apache/camel/component/seda/SedaConsumer.java:
##########
@@ -215,15 +210,38 @@ protected void doRun() {
         }
     }
 
+    /**
+     * Strategy to invoke when the exchange is done being processed.
+     * <p/>
+     * This method is meant to be overridden by subclasses to be able to mimic 
the behavior of the legacy component
+     * camel-vm, that is why the parameter {@code prepared} is not used by 
default.
+     *
+     * @param original the exchange before being processed
+     * @param prepared the exchange processed
+     */
+    protected void onProcessingDone(Exchange original, Exchange prepared) {

Review Comment:
   You do not need to have 2 parameters here, it can just be named exchange, 
and it should be the `prepared` exchange that is being called from the callback.



-- 
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]

Reply via email to