cnauroth commented on PR #7368: URL: https://github.com/apache/hadoop/pull/7368#issuecomment-2646894137
> > +1, with one minor change requested. > > Additionally, is there a JUnit 4 dependency that should be removed from hadoop-distcp/pom.xml? > > @cnauroth We cannot remove the JUnit4 dependency from the `pom` file yet because `hadoop-distcp` has two test dependencies on `hadoop-common`. I am still following up on this in #7375, but it does not affect PR #7368. My plan is to try removing the JUnit4 dependencies from some modules once we complete the upgrade of the `hadoop-common` dependency. Oh, that makes perfect sense. Thank you! -- 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]
