This is an automated email from the ASF dual-hosted git repository.

Jackie-Jiang 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 75bebdf1f2e Fix port reuse in ServerTableSizeReaderTest (#20218)
75bebdf1f2e is described below

commit 75bebdf1f2ec87032ac733bf6414390e68dc624f
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Tue Nov 10 16:07:48 2026 -0800

    Fix port reuse in ServerTableSizeReaderTest (#20218)
---
 .../org/apache/pinot/controller/api/ServerTableSizeReaderTest.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/pinot-controller/src/test/java/org/apache/pinot/controller/api/ServerTableSizeReaderTest.java
 
b/pinot-controller/src/test/java/org/apache/pinot/controller/api/ServerTableSizeReaderTest.java
index 23b79da7dcc..8ef9bad0400 100644
--- 
a/pinot-controller/src/test/java/org/apache/pinot/controller/api/ServerTableSizeReaderTest.java
+++ 
b/pinot-controller/src/test/java/org/apache/pinot/controller/api/ServerTableSizeReaderTest.java
@@ -86,10 +86,11 @@ public class ServerTableSizeReaderTest {
     _tableInfo3 = createTableSizeInfo(TABLE_NAME, new ArrayList<Integer>());
     startServer(0, createHandler(200, _tableInfo1, 0));
     startServer(1, createHandler(200, _tableInfo2, 0));
-    _endpointList.set(2, endpoint(findUnusedPort()));
     startServer(3, createHandler(500, null, 0));
     startServer(4, createHandler(200, null, TIMEOUT_MSEC + 1000));
     startServer(5, createHandler(200, _tableInfo3, 0));
+    // Choose the unused port last so a mock server cannot reuse it after 
findUnusedPort() releases it.
+    _endpointList.set(2, endpoint(findUnusedPort()));
   }
 
   @AfterClass


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to