nandorKollar commented on issue #14310: URL: https://github.com/apache/iceberg/issues/14310#issuecomment-3397122054
I suspect that the problem here is that when you build the Parquet writer, the builder falls back to the deprecated `ParquetWriteAdapter`. Checked why tests don't fail with NPE, and it seems that each test case builds the writer with either `GenericParquetWriter::create` or `InternalWriter::createWriter`, which means that the fallback option, `ParquetWriteAdapter` it not tested at all. Since the `ParquetWriteAdapter` is marked as deprecated, I don't think it makes much sense to fix it. I think that instead the default option (when one doesn't use writer function builder parameter) might need to be changed, maybe to `GenericParquetWriter::create`? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
