alamb commented on PR #11157: URL: https://github.com/apache/arrow-rs/pull/11157#issuecomment-6005450076
> I think this is in good shape now. The outstanding issues are: > > * Do we want delay until we can make breaking changes and simplify the selection API? If we delay it is going to be like 4 months would be my prediction. I vote for the less elegant but backwards compatible API and a ticket to clean it up (along with inline comments referencing it) > * Do we agree that replace when calling `read_page_indexes` repeatedly is the way to go? I think it is ok as written in this PR as it is no worse (practically) than today. Longer term I think we should come up with an example of how someone would incrementally load parts of the page index "on demand" -- maybe we should pull it into its own "PageIndexParser" or something (and similarly for BloomFilter ...) 🤔 -- 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]
