Repository: zeppelin Updated Branches: refs/heads/master 48647e597 -> 6fc3057d5
[MINOR] Verify appId is not null in LivyInterpreterIT ### What is this PR for? Someone complain that they could not get appId, this PR just try to verify appId returned by livy rest api is not null. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? * No jira created ### 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 <zjf...@apache.org> Closes #2999 from zjffdu/minor_livy and squashes the following commits: eae0cf58c [Jeff Zhang] [MINOR] Verify appId is null in LivyInterpreterIT Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/6fc3057d Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/6fc3057d Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/6fc3057d Branch: refs/heads/master Commit: 6fc3057d529691e697361ef9a67e21cac75e21ed Parents: 48647e5 Author: Jeff Zhang <zjf...@apache.org> Authored: Fri Jun 1 21:58:40 2018 +0800 Committer: Jeff Zhang <zjf...@apache.org> Committed: Thu Jun 7 15:23:39 2018 +0800 ---------------------------------------------------------------------- .../src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/zeppelin/blob/6fc3057d/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java ---------------------------------------------------------------------- diff --git a/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java b/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java index 5deb52d..76e2103 100644 --- a/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java +++ b/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java @@ -495,7 +495,8 @@ public class LivyInterpreterIT { InterpreterResult result = sparkInterpreter.interpret("sc.version", context); assertEquals(InterpreterResult.Code.SUCCESS, result.code()); assertEquals(2, result.message().size()); - assertTrue(result.message().get(1).getData().contains("Spark Application Id")); + // check yarn appId and ensure it is not null + assertTrue(result.message().get(1).getData().contains("Spark Application Id: application_")); // html output String htmlCode = "println(\"%html <h1> hello </h1>\")";