alamb opened a new pull request, #21872: URL: https://github.com/apache/datafusion/pull/21872
## Which issue does this PR close? - N/A (documentation-only change). ## Rationale for this change The existing doc comments describing the relationship between ExecutionProps and TaskContext were vague and, in a few places, overstated the distinction. In particular, terms like "execution-facing" were hard to interpret, and some wording implied invariants that are not actually required by the current APIs. ## What changes are included in this PR? - Update the doc comments on ExecutionProps and TaskContext - Describe ExecutionProps in terms of logical optimization and physical-plan / physical-expression construction - Describe TaskContext as the runtime context passed to physical operators during physical-plan execution - Clarify why the two structures remain separate ## Are these changes tested? Not with automated tests. This is a comment-only change. ## Are there any user-facing changes? No. -- 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]
