dongjoon-hyun commented on pull request #35267: URL: https://github.com/apache/spark/pull/35267#issuecomment-1018318880
It seems that you are ignoring the downstream or user codes which might catch `ChunkFetchFailureException`. This PR will lead a compilation error or ClassNotFound error in Apache Spark 3.3 in those downstream code. It could be considered as a regression in the user perspective. > Yes, the class is only used here. In principle, Apache Spark has a deprecation process for the all user facing changes to avoid any surprise in the production. Please take a look at the community policy. - https://spark.apache.org/versioning-policy.html FYI, historically, Apache Spark community tried to clean up many things at 3.0 and later recovered back many things to avoid those kind of hassle. For this one, I'm not sure about the benefit. Is this worth of the removal cost? -- 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]
