sumeetgajjar commented on code in PR #4465:
URL: https://github.com/apache/iceberg/pull/4465#discussion_r842199986
##########
.baseline/checkstyle/checkstyle.xml:
##########
@@ -122,7 +122,8 @@
org.apache.parquet.schema.PrimitiveType.PrimitiveTypeName.*,
org.apache.spark.sql.functions.*,
org.apache.spark.sql.connector.iceberg.write.RowLevelOperation.Command.*,
- org.junit.Assert.*"/>
+ org.junit.Assert.*,
+ org.assertj.core.api.Assertions.*"/>
Review Comment:
Hi Ryan - without the above change, the `checkstyleIntegration` task fails
with the following error:
```
[ant:checkstyle] [ERROR]
/<redacted>/upstream-iceberg/azure/src/integration/java/org/apache/iceberg/azure/blob/TestAzureBlobOutputStream.java:38:46:
Using a static member import should be avoided -
org.assertj.core.api.Assertions.assertThat. [AvoidStaticImport]
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':iceberg-azure:checkstyleIntegration'.
> Checkstyle rule violations were found. See the report at:
file:///<redacted>/upstream-iceberg/azure/build/reports/checkstyle/integration.html
Checkstyle files with violations: 5
Checkstyle violations by severity: [error:12]
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
