alamb commented on issue #7363:
URL: https://github.com/apache/arrow-rs/issues/7363#issuecomment-2855258588

   > Thank you [@XiangpengHao](https://github.com/XiangpengHao) for double 
check, i tried 2 data pages, it's still regression, and 3 data pages it's ok, 3 
pages caching is the smallest for Q27 with no regression, so i add 3 data pages 
to the above commit. I think it's enough for most cases.
   
   Maybe we can change the cache so it is based on row count (aka caches enough 
pages for the total output record batch) rather than some fixed number


-- 
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