findepi commented on code in PR #10474:
URL: https://github.com/apache/iceberg/pull/10474#discussion_r1639613179


##########
aws/src/integration/java/org/apache/iceberg/aws/glue/TestGlueCatalogCommitFailure.java:
##########
@@ -531,7 +531,8 @@ private int metadataFileCount(TableMetadata metadata) {
                         new 
File(S3TestUtil.getKeyFromUri(metadata.metadataFileLocation()))
                             .getParent())
                     .build())
-            .contents().stream()
+            .contents()
+            .stream()

Review Comment:
   > the formatting changes are most likely a result of updating the GJF version
   
   yes, and that's why they come in separate commit
   
   ```
   Update spotless to run on Java 21 
   
   This involves code formatting changes to comply with updated formatting
   library.
   ```
   
   
   > (which we can't update as long as we build/release with JDK8)
   
   
   this is discussed at 
https://github.com/apache/iceberg/pull/10474#discussion_r1639514171



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to