dongjoon-hyun commented on PR #53217: URL: https://github.com/apache/spark/pull/53217#issuecomment-3577709514
> > I don't think we need this PR, do we? > > @dongjoon-hyun It helps avoid potential backward-compatibility issues if external code extends the `LogKey` trait in Spark 4.0. It would be good to address this here. +1 for your passion. However, I don't think that's the Apache Spark community API contract. Moreover, your claim is very misleading because the Scala version is also different between 4.0 and 4.1. I don't think we can claim that 3rd-party libraries compiled with Scala 2.13.16 can work with Spark 4.1 (on Scala 2.13.17) without any issue. Given that, let's not do this. -- 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]
