adriangb commented on PR #18739: URL: https://github.com/apache/datafusion/pull/18739#issuecomment-3621307274
@milenkovicm in order to help determine the best path forward it would be helpful to understand where the pushback against this change is coming from. The conversation has jumped around a bit, but some of the points you brought up are or I could see being worth discussing are: - Code churn this might impose on Ballista - You think the same functionality could have been achieved in some other way (you mentioned using a custom query planner) - The technical details of the implementation (e.g. you would have preferred `SessionConfig` be passed in directly) Or is it something else? Is there a specific one of these that is most worth discussing? -- 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]
