This is an automated email from the ASF dual-hosted git repository. moon pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/zeppelin.git
The following commit(s) were added to refs/heads/master by this push: new df5933d [ZEPPELIN-3984] Flaky test NotebookRestApiTest.testRunWithServerRestart df5933d is described below commit df5933d7f99f6d647f23b8e1ed9294a8b5cb563e Author: Lee moon soo <m...@apache.org> AuthorDate: Thu Sep 19 15:32:43 2019 -0700 [ZEPPELIN-3984] Flaky test NotebookRestApiTest.testRunWithServerRestart ### What is this PR for? This PR trying to fix flaky test described in https://issues.apache.org/jira/browse/ZEPPELIN-3984 ### What type of PR is it? Improvement ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3984 ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: Lee moon soo <m...@apache.org> Closes #3456 from Leemoonsoo/ZEPPELIN-3984 and squashes the following commits: 592ad4abb [Lee moon soo] try not make NPE --- .../src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java b/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java index 079fe5f..bc3bc04 100644 --- a/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java +++ b/zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java @@ -422,7 +422,7 @@ public class NotebookRestApiTest extends AbstractTestRestApi { assertEquals(resp.get("status"), "OK"); post2.releaseConnection(); - assertEquals(p1.getReturn().toString(), Job.Status.FINISHED, p1.getStatus()); + assertEquals(Job.Status.FINISHED, p1.getStatus()); assertEquals(p2.getReturn().toString(), Job.Status.FINISHED, p2.getStatus()); assertNotNull(p2.getReturn()); assertEquals("abc\n", p2.getReturn().message().get(0).getData());