XiangpengHao commented on PR #7850:
URL: https://github.com/apache/arrow-rs/pull/7850#issuecomment-3075979442

   > So one thing I didn't understand after reading this PR in detail was how 
the relative row positions are updated after applying a filter.
   
   Thank you for reviewing this @alamb !
   
   This is place we do merge back filter selection to original selection: 
https://github.com/XiangpengHao/arrow-rs/blob/5537bcb0870ba21549e72b58b65237ba823eec50/parquet/src/arrow/arrow_reader/read_plan.rs#L119
   
   i.e., this pr does not change how we represent selections, we still use the 
existing implementation, the only difference is that we added a transparent 
cache layer, rest of the code should all be the same.
   
   
   
   


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to