stevenzwu commented on pull request #2566: URL: https://github.com/apache/iceberg/pull/2566#issuecomment-835118192
@zhangjun0x01 I am very interested in this PR and the general support of vectorized reader. While working on FLIP-27 Flink source [1], one of the open question is if we should extend from Flink's `FileSourceSplit` and `BulkFormat` for vectorized reader [2]. Now if we are making the effort in Iceberg to support vectorized readers for Orc and Parquet. Then I think we shouldn't need to extend from Flink's vectorized reader. cc @openinx [1] the uber PR for FLIP-27 Iceberg source that are broken down into smaller PRs: https://github.com/apache/iceberg/pull/2105 [2] PR for the split reader: https://github.com/apache/iceberg/pull/2305 -- 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]
