Repository: zeppelin Updated Branches: refs/heads/branch-0.8 81579da87 -> 32698dc19
[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 (cherry picked from commit 6fc3057d529691e697361ef9a67e21cac75e21ed) Signed-off-by: Jeff Zhang <zjf...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/32698dc1 Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/32698dc1 Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/32698dc1 Branch: refs/heads/branch-0.8 Commit: 32698dc19582a46926456d954c3df58b592f7f0e Parents: 81579da 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:50 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/32698dc1/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 c7fbc8d..fd3c253 100644 --- a/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java +++ b/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java @@ -491,7 +491,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>\")";
