gnodet commented on code in PR #24713:
URL: https://github.com/apache/camel/pull/24713#discussion_r3602205715
##########
core/camel-base-engine/src/main/java/org/apache/camel/impl/debugger/BacklogTracer.java:
##########
@@ -62,8 +62,12 @@ public class BacklogTracer extends ServiceSupport implements
org.apache.camel.sp
public static final int MAX_BACKLOG_SIZE = 1000;
private final CamelContext camelContext;
private final Language simple;
- private boolean enabled;
- private boolean standby;
+ // These three flags are toggled at runtime via JMX/management APIs while
routing
Review Comment:
_Claude Code on behalf of gnodet_
Good point -- the comment was vague because `activityEnabled` sits 25 lines
below `enabled`/`standby`, so a reader cannot tell which "three flags" the
comment refers to.
Fixed in 0fafabc3feb0: the comment now explicitly names the three fields
(`enabled`, `standby`, and `activityEnabled (further below)`) and a
cross-reference comment was added next to `activityEnabled` pointing back to
the explanation above.
--
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]
