This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 14333f3f927 Add missing pinot-java-client test dependency to
pinot-broker (#19162)
14333f3f927 is described below
commit 14333f3f927b165caa9affa8ff656ca5aa9c86d9
Author: Yash Mayya <[email protected]>
AuthorDate: Wed Aug 5 14:06:01 2026 -0400
Add missing pinot-java-client test dependency to pinot-broker (#19162)
---
pinot-broker/pom.xml | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/pinot-broker/pom.xml b/pinot-broker/pom.xml
index a165f33db17..b96db348c40 100644
--- a/pinot-broker/pom.xml
+++ b/pinot-broker/pom.xml
@@ -78,6 +78,17 @@
<type>test-jar</type>
<scope>test</scope>
</dependency>
+ <!--
+ Required by ControllerTest in the pinot-controller test-jar above, which
uses PinotAdminClient.
+ pinot-controller declares pinot-java-client at test scope, and
test-scoped dependencies are not
+ transitive, so it does not reach pinot-broker's test classpath. The
other consumers of the
+ controller test-jar already have it, either declared directly or via
pinot-integration-test-base.
+ -->
+ <dependency>
+ <groupId>org.apache.pinot</groupId>
+ <artifactId>pinot-java-client</artifactId>
+ <scope>test</scope>
+ </dependency>
<dependency>
<groupId>org.apache.pinot</groupId>
<artifactId>pinot-query-planner</artifactId>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
