From commits-return-4213-archive-asf-public=cust-asf.ponee.io@zeppelin.apache.org Tue Mar 6 05:52:41 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 38D85180677 for ; Tue, 6 Mar 2018 05:52:41 +0100 (CET) Received: (qmail 99227 invoked by uid 500); 6 Mar 2018 04:52:39 -0000 Mailing-List: contact commits-help@zeppelin.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@zeppelin.apache.org Delivered-To: mailing list commits@zeppelin.apache.org Received: (qmail 98975 invoked by uid 99); 6 Mar 2018 04:52:39 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Mar 2018 04:52:39 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 25AFFF6440; Tue, 6 Mar 2018 04:52:39 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: zjffdu@apache.org To: commits@zeppelin.apache.org Message-Id: <596d3f85ecda4e9eb5e292abb11a498f@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: zeppelin git commit: ZEPPELIN-3277. NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent fails Date: Tue, 6 Mar 2018 04:52:39 +0000 (UTC) Repository: zeppelin Updated Branches: refs/heads/master a6cd0e24c -> c77c549e3 ZEPPELIN-3277. NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent fails ### What is this PR for? Trivial fix for flaky test, see the comment for details. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-3277 ### How should this be tested? * CI pass ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Jeff Zhang Closes #2838 from zjffdu/ZEPPELIN-3277 and squashes the following commits: 68827d7 [Jeff Zhang] ZEPPELIN-3277. NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent fails Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/c77c549e Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/c77c549e Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/c77c549e Branch: refs/heads/master Commit: c77c549e3f6bfa7b08ebc0bfaacb2bd8719c875f Parents: a6cd0e2 Author: Jeff Zhang Authored: Mon Mar 5 15:52:14 2018 +0800 Committer: Jeff Zhang Committed: Tue Mar 6 12:52:33 2018 +0800 ---------------------------------------------------------------------- .../test/java/org/apache/zeppelin/socket/NotebookServerTest.java | 2 ++ 1 file changed, 2 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/zeppelin/blob/c77c549e/zeppelin-server/src/test/java/org/apache/zeppelin/socket/NotebookServerTest.java ---------------------------------------------------------------------- diff --git a/zeppelin-server/src/test/java/org/apache/zeppelin/socket/NotebookServerTest.java b/zeppelin-server/src/test/java/org/apache/zeppelin/socket/NotebookServerTest.java index a0cb587..0b68ca6 100644 --- a/zeppelin-server/src/test/java/org/apache/zeppelin/socket/NotebookServerTest.java +++ b/zeppelin-server/src/test/java/org/apache/zeppelin/socket/NotebookServerTest.java @@ -123,6 +123,8 @@ public class NotebookServerTest extends AbstractTestRestApi { } Thread.sleep(100); } + // sleep for 1 second to make sure job running thread finish to fire event. See ZEPPELIN-3277 + Thread.sleep(1000); // add angularObject interpreterGroup.getAngularObjectRegistry().add("object1", "value1", note1.getId(), null);