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) <[email protected]> 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: [email protected] For additional commands, e-mail: [email protected]
