stoty commented on PR #8062: URL: https://github.com/apache/hadoop/pull/8062#issuecomment-3656220004
I don't like that approach @pan3793 . I've seen some of the earlier hacks that were made in order to fix some specific Spark tests, and they are not nice. Fixing Hadoop just enough that the Spark test suite passes is fragile, and will lead to future problems, even when using Spark. We already know how to get Hadoop to work with Java 25, and we already take all the complexity and risk (not that I think it's risky) by backporting this patch. The right thing to do would be adding full Java 25 support with CI pipeline, which would catch any regressions. (there is no Java 25 CI pipeline for trunk yet, either) -- 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]
