tomtongue commented on code in PR #12945:
URL: https://github.com/apache/iceberg/pull/12945#discussion_r2069946426


##########
spark/v3.4/spark/src/test/java/org/apache/iceberg/spark/sql/TestPartitionedWrites.java:
##########
@@ -18,12 +18,4 @@
  */
 package org.apache.iceberg.spark.sql;
 
-import java.util.Map;
-
-public class TestPartitionedWrites extends PartitionedWritesTestBase {
-
-  public TestPartitionedWrites(
-      String catalogName, String implementation, Map<String, String> config) {
-    super(catalogName, implementation, config);
-  }
-}
+public class TestPartitionedWrites extends PartitionedWritesTestBase {}

Review Comment:
   Can we remove this test (and the test in Spark 3.5)? I believe this test 
will be used as the super-class for other related tests, but this is not 
actually used now. For now, I keep this test for further use.



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