Skip to content

PR #15810 #2240

Sign in to view logs

PR #15810

PR #15810 #2240

Triggered via dynamic January 21, 2025 15:59
@ritwiksahaniritwiksahani
⁠ 7e3df02
refs/pull/15810/head
Status Success
Total duration 1m 42s
Artifacts –

codeql

on: dynamic
Matrix: analyze
Fit to window
Zoom out
Zoom in

Annotations

1 error and 12 warnings
CoreSchedulerServiceTest.testProgramEvents: CoreSchedulerServiceTest#L421
Timeout occurred. Expected 1 but found 0.
Analyze (python)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Analyze (ruby)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Analyze (javascript)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/runtime/schedule/trigger/TimeTrigger.java#L141
All overloaded methods should be placed next to each other. Placing non-overloaded methods in between overloaded methods with the same type is a violation. Previous overloaded method located at line '57'.
com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/runtime/schedule/trigger/NotificationContext.java#L47
Missing a Javadoc comment.
com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/runtime/schedule/queue/JobQueueTable.java#L63
Empty line should be followed by <p> tag on the next line.
com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/runtime/schedule/queue/JobQueueTable.java#L69
Empty line should be followed by <p> tag on the next line.
com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/runtime/schedule/queue/JobQueueTable.java#L76
Empty line should be followed by <p> tag on the next line.
com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/runtime/schedule/queue/JobQueueTable.java#L105
Missing a Javadoc comment.
com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/runtime/schedule/queue/JobQueueTable.java#L437
First sentence of Javadoc is missing an ending period.
com.puppycrawl.tools.checkstyle.checks.NewlineAtEndOfFileCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/runtime/schedule/trigger/ProgramStatusTriggerTest.java#L1
File does not end with a newline.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/runtime/schedule/trigger/ProgramStatusTriggerTest.java#L66
Distance between variable 'notificationList' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).