allthingssecurity commented on code in PR #28382:
URL: https://github.com/apache/camel/pull/28382#discussion_r4370089474
##########
core/camel-core-processor/src/main/java/org/apache/camel/processor/TotalRequestsThrottler.java:
##########
@@ -161,7 +162,7 @@ public boolean process(final Exchange exchange, final
AsyncCallback callback) {
if (state == State.ASYNC) {
if (LOG.isTraceEnabled()) {
- long queuedTime = System.currentTimeMillis() -
queuedStart;
+ long queuedTime = TimeSource.currentTimeMillis() -
queuedStart;
Review Comment:
Fixed in this PR after all, in 13c1c62d670c (at davsclaus's request):
`processAsynchronously` now stores `TimeSource.currentTimeMillis()` in
`CamelThrottlerExchangeQueuedTimestamp`, so both "Queued for {}ms" lines
(`:148` and `:165`) subtract milliseconds from milliseconds. Nothing else reads
that property. The follow-up I mentioned is dropped.
_Claude Code on behalf of allthingssecurity_
--
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]
