pvary commented on code in PR #10956: URL: https://github.com/apache/iceberg/pull/10956#discussion_r1724763903
########## flink/v1.20/flink/src/test/java/org/apache/iceberg/flink/source/TestIcebergSourceBoundedGenericRecord.java: ########## @@ -69,12 +70,13 @@ public class TestIcebergSourceBoundedGenericRecord { private static final HadoopCatalogExtension CATALOG_EXTENSION = new HadoopCatalogExtension(TestFixtures.DATABASE, TestFixtures.TABLE); - @Parameters(name = "format={0}, parallelism = {1}") + @Parameters(name = "format={0}, parallelism = {1}, useConverter = {2}") public static Object[][] parameters() { return new Object[][] { - {FileFormat.AVRO, 2}, - {FileFormat.PARQUET, 2}, - {FileFormat.ORC, 2} + {FileFormat.AVRO, 2, false}, + {FileFormat.PARQUET, 2, true}, + {FileFormat.PARQUET, 2, false}, + {FileFormat.ORC, 2, true} Review Comment: Why did you opt for this exact parametrization? I happy for having tests for Parquet in both cases, as that is the main usecase for us now: ``` {FileFormat.PARQUET, 2, true}, {FileFormat.PARQUET, 2, false}, ``` I might have opted for testing the future features, and keep one test for backward comp, like this: ``` {FileFormat.AVRO, 2, true}, {FileFormat.PARQUET, 2, true}, {FileFormat.PARQUET, 2, false}, {FileFormat.ORC, 2, true} ``` -- 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