zhuqi-lucas commented on PR #23696: URL: https://github.com/apache/datafusion/pull/23696#issuecomment-5264293730
Thanks @alamb for the review and the good suggestions! Addressed all of them in e0fbd24 (replies inline): wrapped the prebuilt list in a `PrebuiltRowFilterCandidateList`, reframed the doc as a cache, pulled the boundary plan-manipulation into `sync_rg_plan_to_decoder_frontier` / `advance_rg_plan_to` / `prune_boundary_row_groups`, and trimmed the over-detailed comments. Left the rebuild+toggle block inline for now — happy to extract it as a follow-up. -- 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]
