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:31 UTC
[incubator-pinot] branch fix-flaky-test created (now 379193b)
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-flaky-test
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 379193b Try fixing flaky tests by adding 0.5s sleep
This branch includes the following new commits:
new 379193b Try fixing flaky tests by adding 0.5s sleep
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[incubator-pinot] 01/01: Try fixing flaky tests by adding 0.5s sleep
Posted by jl...@apache.org.
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