You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by jl...@apache.org on 2019/01/08 21:43:32 UTC
[incubator-pinot] 01/01: Try fixing flaky tests by adding 0.5s sleep
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch fix-flaky-test
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 379193b8db8f09b95c482f1c2ffa322028ae423c
Author: Jack Li(Analytics Engineering) <jl...@jlli-mn1.linkedin.biz>
AuthorDate: Tue Jan 8 13:43:08 2019 -0800
Try fixing flaky tests by adding 0.5s sleep
---
.../controller/api/resources/PinotInstanceRestletResourceTest.java | 3 +++
1 file changed, 3 insertions(+)
diff --git a/pinot-controller/src/test/java/com/linkedin/pinot/controller/api/resources/PinotInstanceRestletResourceTest.java b/pinot-controller/src/test/java/com/linkedin/pinot/controller/api/resources/PinotInstanceRestletResourceTest.java
index e527708..e2b986e 100644
--- a/pinot-controller/src/test/java/com/linkedin/pinot/controller/api/resources/PinotInstanceRestletResourceTest.java
+++ b/pinot-controller/src/test/java/com/linkedin/pinot/controller/api/resources/PinotInstanceRestletResourceTest.java
@@ -63,6 +63,9 @@ public class PinotInstanceRestletResourceTest extends ControllerTest {
serverInstance.put("host", "2.3.4.5");
sendPostRequest(_controllerRequestURLBuilder.forInstanceCreate(), serverInstance.toString());
+ // Sleep 0.5s for cache accessor to update cache.
+ Thread.sleep(500L);
+
// Check that there are four instances
instanceList = new JSONObject(sendGetRequest(_controllerRequestURLBuilder.forInstanceList()));
assertEquals(instanceList.getJSONArray("instances").length(), 4,
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org