ammachado commented on code in PR #28819:
URL: https://github.com/apache/camel/pull/28819#discussion_r4507914421


##########
core/camel-support/src/main/java/org/apache/camel/support/ExtendedExchangeExtension.java:
##########
@@ -42,6 +42,7 @@ public class ExtendedExchangeExtension implements 
ExchangeExtension {
     private boolean streamCacheDisabled;
     private boolean redeliveryExhausted;
     private int redeliveryCounter = -1;
+    private long lastNodeEndNanos;

Review Comment:
   Thanks. I went the other way: 31dabf42e349 removes the field and the 
overrides from `ExtendedExchangeExtension` instead of adding a reset.
   
   Nothing creates an `ExtendedExchangeExtension` any more: `AbstractExchange` 
holds those fields itself and `getExchangeExtension()` returns `this`, and the 
only other mentions of the class are two comments in `AbstractExchange`. The 
`default` methods on `ExchangeExtension` (return `0`, do nothing) are enough 
there, which is also how `setSharedUnitOfWork` was added for 4.24. The reset 
that matters, `AbstractExchange.resetExtension()`, clears the time and has a 
test.
   
   _Claude Code on behalf of Adriano Machado (@ammachado)_
   
   _This was generated by an AI agent and may contain inaccuracies. Please 
verify before relying on it._



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