adriangb commented on issue #11234: URL: https://github.com/apache/arrow-rs/issues/11234#issuecomment-5936485117
I simplified the stack to make review easier. It is now 4 PRs instead of 7, and the issue description has the new plan: - **Ready now, in parallel:** https://github.com/apache/arrow-rs/pull/11235 (now also contains https://github.com/apache/arrow-rs/pull/11236) and https://github.com/apache/arrow-rs/pull/11237 (benchmark bot: `Nbuf/100000ranges` 15× faster). - **Draft until 11235 merges:** https://github.com/apache/arrow-rs/pull/11238. - **Draft until 11238 merges:** https://github.com/apache/arrow-rs/pull/11240 (now also contains https://github.com/apache/arrow-rs/pull/11239 and https://github.com/apache/arrow-rs/pull/11241). The page release and the predicate cache must agree on when cached pages are released, so they are one PR. The merge notes in my earlier comment no longer apply. The only remaining one: whichever of 11235 and 11237 merges second gets a small rebase. -- 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]
