XiangpengHao commented on PR #11587: URL: https://github.com/apache/datafusion/pull/11587#issuecomment-2242943692
> This logic (batch in, batch out) should be a separate helper function (maybe living somewhere else, as it could be useful in other contexts too). Agree! I moved the logic to a separate function. I tentatively put it right after the `CoalesceBatchesStream` implementation, please let me know if there's a better way to organize it! -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
