binmahone commented on PR #38867: URL: https://github.com/apache/arrow/pull/38867#issuecomment-1828959046
hi @emkornfield , thanks for your thorough review. Looks like I should have provided some background to reviewers: 1. We're working on native parquet reading in Clickhouse, which maintains a arrow-13 fork. That's why this PR draft is based on arrow-13. 2. It's my first time to contribute to Arrow, so I'm consulting @wgtmac and @mapleFU offline to check whether this PR is needed by Arrow community and how to revise the PR to meet community standard. 3. @wgtmac has quickly summarized a detailed design doc at https://docs.google.com/document/d/1SeVcYudu6uD9rb9zRAnlLGgdauutaNZlAaS0gVzjkgM. When the design doc has reached consensus, I'll come back to this PR and address all the review comments. Looking forward to contributing. Thanks! -- 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]
