milenkovicm opened a new issue, #17596: URL: https://github.com/apache/datafusion/issues/17596
### Is your feature request related to a problem or challenge? With #16970 `SessionContext` has been propagated as parameter of most methods in physical plan proto apart from `PhysicalExtensionCodec::try_decode` which still uses `registry: &dyn FunctionRegistry,`. This is a bit of a problem when `try_decode` calls other methods in physical plan proto as there is no ctx to provide as parameter ### Describe the solution you'd like I would argue that SessionContext is an overkill and that all methods should align on `TaskContext` which holds all required information. ### Describe alternatives you've considered Changing `PhysicalExtensionCodec::try_decode` registry to SessionContext has been considered, but i would argue SessionContext is to wide interface to be used here ### Additional context This issue has arisen when updating ballista to datafusion 50. Part of the issue is that decoder has been invoked on executor side where only TaskContext can be provided -- 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]
