Skip to content

PR #15800

PR #15800 #2249

GitHub Actions / Checkstyle completed Jan 22, 2025 in 0s

Checkstyle report

64 violation(s) found

Annotations

Check warning on line 24 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/events/StartProgramEventSubscriber.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'io.cdap.cdap.internal.app.services.FlowControlService' import. Should be before 'io.cdap.cdap.internal.app.services.ProgramLifecycleService'.

Check warning on line 114 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/FlowControlService.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 98 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/ProgramNotificationSubscriberService.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

First sentence of Javadoc is missing an ending period.

Check warning on line 247 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/ProgramNotificationSubscriberService.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.design.OneTopLevelClassCheck

Top-level class ProgramNotificationSingleTopicSubscriberService has to reside in its own source file.

Check warning on line 918 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/ProgramNotificationSubscriberService.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.MissingSwitchDefaultCheck

switch without "default" clause.

Check warning on line 1548 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/ProgramLifecycleService.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'appCDAPVersion' must contain no more than '1' consecutive capital letters.

Check warning on line 1 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.sizes.FileLengthCheck

File length is 3,137 lines (max allowed is 3,000).

Check warning on line 104 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

Empty line should be followed by <p> tag on the next line.

Check warning on line 108 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

Empty line should be followed by <p> tag on the next line.

Check warning on line 111 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

Empty line should be followed by <p> tag on the next line.

Check warning on line 113 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

Empty line should be followed by <p> tag on the next line.

Check warning on line 116 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

Empty line should be followed by <p> tag on the next line.

Check warning on line 118 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

Empty line should be followed by <p> tag on the next line.

Check warning on line 395 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 414 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 451 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 794 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 1252 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 1329 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'meta' declaration and its first usage is 6, 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).

Check warning on line 1784 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

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 '1,692'.

Check warning on line 1866 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

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 '1,823'.

Check warning on line 1881 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

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 '1,758'.

Check warning on line 1921 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

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 '1,866'.

Check warning on line 1955 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

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 '1,668'.

Check warning on line 1983 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.