attilapiros commented on pull request #30763: URL: https://github.com/apache/spark/pull/30763#issuecomment-800791087
We are all agree more abstraction here is really a good idea and reading https://github.com/apache/spark/pull/30763#issuecomment-792865534 gives me the impression we both worry about the impact of the change but as I see you have solutions for all the concerns: https://github.com/apache/spark/pull/30763#issuecomment-792861035. @Ngone51 I am fine if you proceed and when it is ready we can see the real price of this change. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
