xiangfu0 commented on code in PR #20112:
URL: https://github.com/apache/pinot/pull/20112#discussion_r4486102594
##########
pom.xml:
##########
@@ -660,6 +660,14 @@
<dependencyManagement>
<dependencies>
+ <!-- Keep Kafka's testkit and the TestNG platform provider on the same
JUnit infrastructure. -->
+ <dependency>
+ <groupId>org.junit</groupId>
+ <artifactId>junit-bom</artifactId>
Review Comment:
Fixed in 10fe73e086. Moved the JUnit BOM beside the other test BOMs and
removed the redundant managed Jupiter API/Platform Suite entries. Effective
models retain all 16 JUnit artifact versions at 6.1.3, and remaining direct
Suite declarations explicitly use test scope. Kept the TestNG/Platform
explanation and removed the obsolete readiness comment.
--
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]