adriangb commented on PR #25339: URL: https://github.com/apache/datafusion/pull/25339#issuecomment-5783384790
@kosiew @jayzhan211 I split this PR to make review easier. After several rounds it'd gotten quite large and accumulated several different dimensions of work. The tests landed in [apache/datafusion#25558](https://github.com/apache/datafusion/pull/25558), this PR is now only the executor fix, and the optimizer fix moved to [apache/datafusion#25560](https://github.com/apache/datafusion/pull/25560), stacked on this one. The executor code is the same as at 78b49ba214, which which was already reviewed, less one dead line. Your approvals were for the combined change, so could you confirm they still hold for this scope? The optimizer threads are fixed in [apache/datafusion#25560](https://github.com/apache/datafusion/pull/25560). Thanks! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
