Skip to content

Address review comments

Sign in for the full log view
GitHub Actions / Checkstyle completed Feb 21, 2025 in 0s

Checkstyle report

16 violation(s) found

Annotations

Check warning on line 43 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTable.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.storage.spanner.compression.Compressor' import. Should be before 'io.cdap.cdap.storage.spanner.compression.CompressorType'.

Check warning on line 72 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTable.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 216 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTable.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 '181'.

Check warning on line 679 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTable.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 781 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTable.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 41 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/compression/SnappyCompressor.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 '29'.

Check warning on line 46 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/compression/SnappyCompressor.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 '35'.

Check warning on line 51 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/compression/Compressor.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 '33'.

Check warning on line 60 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/compression/Compressor.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 '42'.

Check warning on line 1 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/compression/CompressorFactory.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.NewlineAtEndOfFileCheck 

File does not end with a newline.

Check warning on line 42 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStorageProvider.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 29 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredRow.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.storage.spanner.compression.Compressor' import. Should be before 'io.cdap.cdap.storage.spanner.compression.CompressorType'.

Check warning on line 46 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTableSchema.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck 

'?' should be on a new line.

Check warning on line 50 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTableSchema.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

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

Summary javadoc is missing.

Check warning on line 123 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTableAdmin.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

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

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

Check warning on line 160 in cdap-storage-ext-spanner/src/main/java/io/cdap/cdap/storage/spanner/SpannerStructuredTableAdmin.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

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

Missing a Javadoc comment.
View more details on GitHub Actions