pvary opened a new issue #1924: URL: https://github.com/apache/iceberg/issues/1924
TestHiveIcebergStorageHandler currently runs the full matrix of: - FileFormats - Avro, ORC, Parquet - Execution engines - mr, tez - Catalog types - Hive, Hadoop Catalog, Hadoop Table, Custom catalog Currently there are 32 test cases in the class, this means we have 768 test cases, and in some cases we do not even use the parameters in the tests. We should separate this test at least 3 classes: - Tests which does not write data - only Catalog types should be used as a parameter - Tests which does write data but using only local MR execution (`SELECT * FROM table`) - Catalog types and FileFormats should be used as parameters. We could set the execution engine to `spark` to make sure we do not accidentally put tests here which are using supported execution engines - Tests which are using the engines too - every parameter we can keep ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
