gnodet-bot commented on code in PR #29384:
URL: https://github.com/apache/camel/pull/29384#discussion_r4503022189
##########
dsl/camel-jbang/camel-jbang-plugin-tui/src/main/java/org/apache/camel/dsl/jbang/core/commands/tui/Timeline.java:
##########
@@ -432,6 +495,16 @@ private static long[] filled(int seconds) {
* The causes in the window: the errors per second by label, then the
faults of each chaos rule. Each series is
* oldest first.
*/
+ /** A WARN or ERROR line in the log of the integration, at the second of
its time. */
Review Comment:
💬 **Javadoc displacement (low):** `logLine()` was inserted between the
Javadoc for `causes()` (lines 494–497) and the `causes()` method (line 508),
orphaning that Javadoc. The fix is to move `logLine()` and its own Javadoc
(lines 498–506) above the `causes()` Javadoc block, so the structure reads:
```java
/** A WARN or ERROR line in the log of the integration, at the second of
its time. */
void logLine(String pid, long second, boolean error) {
...
}
/**
* The causes in the window: the errors per second by label, then the
faults of each chaos rule. Each series is
* oldest first.
*/
List<CauseSeries> causes(String pid, long endSecond, int seconds) {
```
--
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]
