alamb opened a new pull request, #12047: URL: https://github.com/apache/datafusion/pull/12047
## Which issue does this PR close? Part of https://github.com/apache/datafusion/issues/7957 and https://github.com/apache/datafusion/issues/11628 ## Rationale for this change I have hopes to improve how coalesce batches works in DataFusion saving a copy to improve performance, partly by avoiding the need for `CoalesceBatchesExec` Part of this is to use the `BatchCoalescer` more, so let's pull it into its own module. This also makes the code boundaries a bit clearer (e.g. the functions that form the public interface now must be marked `pub`) ## What changes are included in this PR? 1. Move `BatchCoalescer` to its own module 2. Refine the documentation ## Are these changes tested? By existing CI <!-- We typically require tests for all PRs in order to: 1. Prevent the code from being accidentally broken by subsequent changes 2. Serve as another way to document the expected behavior of the code If tests are not included in your PR, please explain why (for example, are they covered by existing tests)? --> ## Are there any user-facing changes? No functional changes <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> -- 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
