joseluisll commented on PR #8893: URL: https://github.com/apache/hadoop/pull/8893#issuecomment-6630494014
Thanks @pan3793, good suggestion. I've regrouped the two PRs by kind of change, each covering the whole repository: - #8893 (HADOOP-20015) has only the Java changes: Jetty APIs that Jetty 12 removes, replaced by the JDK, SLF4J or the servlet API. That's Jetty's `Log` in 8 test files across hadoop-common and YARN, the `ConcurrentHashSet` in `FederationInterceptor`, the borrowed logger in `TestDockerCommandExecutor`, and the Jetty cast in `HttpServer2#inferMimeType`. A sweep of the repo also found `MultiException` in `TestLogAggregationService`, now a plain list of failures. Every other Jetty class the code imports still exists in Jetty 12, or moves to the ee8 packages, which belongs to the upgrade itself (HADOOP-19972). - #8894 (HADOOP-20016) has only the pom changes: each module declares the Jetty artifacts it imports directly, at the scope it already resolves them at. The same sweep added a few more modules (hdfs, httpfs, jobclient, NodeManager, Router, web-proxy). The per-module Jetty dependency lists are the same before and after. Neither PR depends on the other. -- 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]
