callmepandey commented on issue #513: URL: https://github.com/apache/iceberg-cpp/issues/513#issuecomment-3758016184
Good catch! I focused on projection/validation but missed that Arrow defaults to `Type::LIST` output regardless of data. We need to expose `ArrowReaderProperties::set_list_type()` through our `ReaderProperties`. Updated scope for this issue: 1. ~~Update `ValidateParquetSchemaEvolution` to accept `LARGE_LIST`~~ (done in current PR) 2. **Add `kListType` to `ReaderProperties`** 3. **Pass through to `arrow_reader_properties.set_list_type()` in `ParquetReader::Impl::Open()`** Let me know if this looks right and I'll update the PR. -- 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]
