rahulsmahadev commented on PR #714: URL: https://github.com/apache/iceberg-cpp/pull/714#issuecomment-4665840212
@wgtmac this implements the design you suggested in #513 — would appreciate a review when you get a chance. One note beyond the two items in the issue: the reader's output Arrow schema (built by `ToArrowSchema`, which uses 32-bit lists) also needed to be aligned so `ProjectRecordBatch` dispatches to the large_list path from #502. I kept that rewrite local to the Parquet reader to avoid changing the `ToArrowSchema` signature, but happy to take a different approach if you'd prefer. -- 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]
