chinmay-bhat commented on code in PR #9342: URL: https://github.com/apache/iceberg/pull/9342#discussion_r1432856893
########## spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/source/TestReadProjection.java: ########## @@ -49,7 +50,7 @@ public abstract class TestReadProjection { protected abstract Record writeAndRead( String desc, Schema writeSchema, Schema readSchema, Record record) throws IOException; - @Rule public TemporaryFolder temp = new TemporaryFolder(); + @TempDir protected Path temp; Review Comment: I chose to reintroduce the imports. For that, I reversed the TempDir change, and changed 1 AssertJ back to Assert (added a TODO so I can change it in the future). Otherwise, the imports were getting auto-removed from the IDE. -- 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