milenkovicm commented on PR #18739: URL: https://github.com/apache/datafusion/pull/18739#issuecomment-3602890951
Lets leave extensions, the change is not very disruptive, I've totally missed part that previous API is still there Would you consider using `OptimizerContext` instead? It’s a very defensive API approach and might not be necessary. It’s been quite stable for a long time, do you expect it to change soon? -- 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]
