zhuqi-lucas commented on PR #186: URL: https://github.com/apache/datafusion-site/pull/186#issuecomment-4998606114
Thanks so much for the deep pass @alamb — the restructure reads great, and the Related Work section is stronger than anything I would've written on my own. Really appreciate the time. I made three tiny tweaks in [1666952](https://github.com/apache/datafusion-site/commit/1666952): - Row-Level Early Stopping paragraph: moved the "skip projection column fetch" attribution to arrow-rs's `RowFilter` (where it actually lives) and clarified that #22450 makes it fire more often by keeping the threshold current. - Dropped "In DataFusion 54" from the conclusion — #22450 merged after 54.0.0 was tagged, so it'll ship in a later release; used "Over the last few DataFusion releases" instead. - Decision-tree figure alt-text: `try_pushdown_sort` → `PushdownSort` for consistency with the prose. Otherwise all your edits LGTM. Happy to go with **Monday July 20** as the publish date — I'll ping Discord over the weekend for any last comments before then. -- 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]
