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 in 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
