lidavidm commented on pull request #10552:
URL: https://github.com/apache/arrow/pull/10552#issuecomment-863620407


   You mean the one in OneShotFragment? I think that's fine.
   
   Should we add the thread pool as a parameter to MakeBackgroundGenerator? So 
that you can't forget. (I guess that'd create some churn once we have 
ARROW-13109 since it would become redundant again.)
   
   This looks like it was a fun issue to debug…


-- 
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:
[email protected]


Reply via email to