PR #15419 #533
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
codeql
on: dynamic
Matrix: Analyze
Annotations
78 warnings
Analyze (javascript)
Failed to restore: Cache service responded with 403
|
|
|
com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L103
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/store/AppMetadataStore.java#L107
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/store/AppMetadataStore.java#L110
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/store/AppMetadataStore.java#L112
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/store/AppMetadataStore.java#L115
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/store/AppMetadataStore.java#L117
Empty line should be followed by <p> tag on the next line.
|
|
|
|
|
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L1247
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).
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L1676
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,612'.
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L1743
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,650'.
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L1762
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,728'.
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L1796
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,588'.
|
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L1861
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,796'.
|
|
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L2156
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,824'.
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L2170
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,861'.
|
|
|
|
|
|
|
com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L2395
First sentence of Javadoc is missing an ending period.
|
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L2481
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 '719'.
|
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/AppMetadataStore.java#L2691
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 '2,126'.
|
com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck:
cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/ApplicationMeta.java#L26
First sentence of Javadoc is missing an ending period.
|
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L117
Distance between variable 'lineageAdmin' declaration and its first usage is 8, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L188
Distance between variable 'lineageAdmin' declaration and its first usage is 9, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L250
Distance between variable 'lineageAdmin' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L282
Distance between variable 'lineageAdmin' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L320
Distance between variable 'lineageAdmin' declaration and its first usage is 13, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L388
Distance between variable 'lineageAdmin' declaration and its first usage is 16, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L469
Distance between variable 'lineageWriter' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L499
Distance between variable 'lineageAdmin' declaration and its first usage is 11, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L512
Distance between variable 'run5' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L643
Distance between variable 'lineageWriter' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L675
Distance between variable 'lineageAdmin' declaration and its first usage is 10, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L688
Distance between variable 'localDataset1' declaration and its first usage is 5, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L689
Distance between variable 'localDataset2' declaration and its first usage is 7, 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).
|
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/LineageAdminTest.java#L788
Abbreviation in name 'workflowIDMap' must contain no more than '1' consecutive capital letters.
|
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/MetadataSubscriberServiceTest.java#L78
Wrong lexicographical order for 'io.cdap.cdap.messaging.context.MultiThreadMessagingContext' import. Should be before 'io.cdap.cdap.messaging.spi.StoreRequest'.
|
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/MetadataSubscriberServiceTest.java#L79
Wrong lexicographical order for 'io.cdap.cdap.messaging.service.CoreMessagingService' import. Should be before 'io.cdap.cdap.messaging.spi.StoreRequest'.
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/MetadataSubscriberServiceTest.java#L174
Distance between variable 'fieldLineageWriter' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/MetadataSubscriberServiceTest.java#L175
Distance between variable 'spark1Run1' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/MetadataSubscriberServiceTest.java#L224
Distance between variable 'fieldLineageReader' declaration and its first usage is 8, 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).
|
SeparatorWrapComma:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/MetadataSubscriberServiceTest.java#L273
',' should be on the previous line.
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/metadata/MetadataSubscriberServiceTest.java#L293
Distance between variable 'workflowRunId' declaration and its first usage is 8, 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).
|
com.puppycrawl.tools.checkstyle.checks.whitespace.EmptyLineSeparatorCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/profile/ProfileServiceTest.java#L91
'METHOD_DEF' should be separated from previous line.
|
com.puppycrawl.tools.checkstyle.checks.whitespace.EmptyLineSeparatorCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/profile/ProfileServiceTest.java#L92
'METHOD_DEF' should be separated from previous line.
|
com.puppycrawl.tools.checkstyle.checks.whitespace.EmptyLineSeparatorCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/profile/ProfileServiceTest.java#L93
'METHOD_DEF' should be separated from previous line.
|
AnnotationLocationMostCases:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/store/NoSqlAppMetadataStoreTest.java#L44
Annotation 'Test' should be alone on line.
|
AnnotationLocationMostCases:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/store/NoSqlAppMetadataStoreTest.java#L49
Annotation 'Test' should be alone on line.
|
AnnotationLocationMostCases:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/store/NoSqlAppMetadataStoreTest.java#L59
Annotation 'Test' should be alone on line.
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/store/AppMetadataStoreTest.java#L659
Distance between variable 'activeRuns' declaration and its first usage is 10, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/store/AppMetadataStoreTest.java#L704
Distance between variable 'activeRuns' declaration and its first usage is 5, 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).
|
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/store/AppMetadataStoreTest.java#L1546
Abbreviation in name 'GSON' must contain no more than '1' consecutive capital letters.
|
com.puppycrawl.tools.checkstyle.checks.naming.LocalVariableNameCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/store/AppMetadataStoreTest.java#L1546
Local variable name 'GSON' must match pattern '^[a-z]([a-zA-Z0-9]+)?$'.
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L117
Distance between variable 'uriInstance' declaration and its first usage is 25, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L120
Distance between variable 'detail1' declaration and its first usage is 9, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L121
Distance between variable 'detail2' declaration and its first usage is 19, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L226
Distance between variable 'uriInstance' declaration and its first usage is 12, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L227
Distance between variable 'uriNamespace1' declaration and its first usage is 13, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L228
Distance between variable 'uriApp' 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L230
Distance between variable 'combinedProperties' declaration and its first usage is 20, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L318
Distance between variable 'uriInstance' declaration and its first usage is 15, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L321
Distance between variable 'uriNamespace2Service' declaration and its first usage is 10, 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).
|
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck:
cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/PreferencesHttpHandlerInternalTest.java#L323
Distance between variable 'programProperties' declaration and its first usage is 7, 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).
|