westonpace commented on pull request #11911:
URL: https://github.com/apache/arrow/pull/11911#issuecomment-997128837


   > But sounds like there are likely already tests that depend on that so it's 
fine?
   
   Yes, there are likely other tests that depend on this but you bring up an 
interesting point.  The scanner can chop large record batches.  We are going to 
be getting more aggressive with this too but probably not to the point where it 
will interfere with these tests.  At the moment it defaults to 1M rows.


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