anmolanmol1234 commented on code in PR #6275: URL: https://github.com/apache/hadoop/pull/6275#discussion_r1416850390
########## hadoop-project/pom.xml: ########## @@ -1288,10 +1288,22 @@ <artifactId>jackson-dataformat-cbor</artifactId> <version>${jackson2.version}</version> </dependency> + <dependency> + <groupId>org.mockito</groupId> + <artifactId>mockito-bom</artifactId> + <version>${mockito.version}</version> + <type>pom</type> Review Comment: I agree with the decision to remove the 'mockito-bom' dependency. However, I've encountered an issue where the DLL files are not being included due to the removal of this dependency. The DLL files are originally included as part of the 'byte-buddy-agent' POM, and 'mockito-core' has a dependency on this module. Despite setting the value of 'mockito-core' as optional in 'hadoop-minicluster,' the transitive dependency is still affecting 'hadoop-client-check-test-invariants,' resulting in an error during the check for jar contents due to disallowed DLL values. Consequently, I've addressed this by adding the necessary expressions to allow these DLL files -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org