geoffreyclaude opened a new pull request, #22711: URL: https://github.com/apache/datafusion/pull/22711
## Which issue does this PR close? - Closes #22708. ## Rationale for this change `EvaluationType::Eager` is documented as describing operators that eagerly generate record batches in spawned Tokio tasks. `BufferExec` and `AnalyzeExec` both drive child input streams from spawned tasks, but their `PlanProperties` currently report lazy evaluation. This makes `EvaluationType` and helpers such as `need_data_exchange(...)` miss these child-polling operators. ## What changes are included in this PR? - Mark `BufferExec` as `EvaluationType::Eager` while keeping its existing cooperative scheduling metadata. - Mark `AnalyzeExec` as `EvaluationType::Eager` in its computed plan properties. ## Are these changes tested? Existing targeted tests pass: ```bash cargo test -p datafusion-physical-plan --lib buffer::tests cargo test -p datafusion-physical-plan --lib analyze::tests ``` I did not add a new test that simply asserts the assigned `EvaluationType`, as that would duplicate the implementation rather than cover behavior. ## Are there any user-facing changes? No direct query-result changes are expected. This updates physical-plan metadata for operators that already drive child streams eagerly. -- 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]
