geoffreyclaude commented on issue #21650:
URL: https://github.com/apache/datafusion/issues/21650#issuecomment-4255064627
> I think @askalt @adriangb @gabotechs @xudong963 @geoffreyclaude
@gene-bordegaray and others may have some thoughts about this topic as well
The main reason I worked on this was `datafusion-tracing`: recursive queries
broke once plans were wrapped, because the runtime wiring depended on concrete
`WorkTableExec` handling. That led to `with_new_state` in #16469 and the
follow-up in #18803.
So I’m sympathetic to moving shared state off plans, but whatever replaces
it should still support wrapper/custom `ExecutionPlan`s without concrete type
checks. `TaskContext` seems like a plausible direction if it preserves that.
--
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]
