ctsk opened a new pull request, #15570: URL: https://github.com/apache/datafusion/pull/15570
Continuing https://github.com/apache/datafusion/pull/15476 and https://github.com/apache/datafusion/pull/15418 > ## Rationale for this change + What changes are included in this PR? > > The logic of HashJoin's `execute` implements differs from the other operators - The recursive call to the `execute` step of the build side are delayed until `collect_left_join` is poll'ed. This PR changes that to make it more alike to the standard implementations of `execute`. > ## Are these changes tested? > > No changes in behaviour are expected. > ## Are there any user-facing changes? > > No Notable changes: * Fixed the bug that caused test failures (partition 0 of the left side was executed multiple times) * Applied the same changes the nested_loop_join and cross_join -- 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
