wjones1 commented on pull request #6979:
URL: https://github.com/apache/arrow/pull/6979#issuecomment-651140872


   Actually @jorisvandenbossche, I agree we should probably just add in the 
batch_size argument (with a sensible default) to those other methods. Took me a 
while to understand what you were getting at, but by providing those parameters 
we avoid the issue where `iter_batches()` calls could mess with the underlying 
parameters when calling the other methods.
   
   I will implement those soon. I would especially welcome input on what the 
docstring should be for the `batch_size` parameter for those other methods 
(what is the user-facing impact of changing it?) and what the default value 
should be.


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

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


Reply via email to