Skip to content

[ZEPPELIN-1624] HeliumApplicationFactoryTest fails on master - #1603

Closed
Leemoonsoo wants to merge 1 commit into
apache:masterfrom
Leemoonsoo:fix_helium_application_factory_test
Closed

[ZEPPELIN-1624] HeliumApplicationFactoryTest fails on master#1603
Leemoonsoo wants to merge 1 commit into
apache:masterfrom
Leemoonsoo:fix_helium_application_factory_test

Conversation

@Leemoonsoo

Copy link
Copy Markdown
Member

What is this PR for?

HeliumApplicationFactoryTest fails on master with following error if i run the test on local machine.

I'm fixing the issue by changing InterpreterFactory loads interpreter properties from interpreter-setting.json file, interpreter-setting.json classresource and Interpreter.register() call

What type of PR is it?

Bug Fix

Todos

  • - Fix the problem

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1624

How should this be tested?

try run HeliumApplicationFactoryTest unittest and see if it successes

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@LeemoonsooLeemoonsoo changed the title [HOTFIX] HeliumApplicationFactoryTest fails on master[ZEPPELIN-1624] HeliumApplicationFactoryTest fails on masterNov 5, 2016
@Leemoonsoo
Leemoonsooforce-pushed the fix_helium_application_factory_test branch from 89b44a3 to e511783CompareNovember 5, 2016 04:10
} catch (Throwable t) {
// nothing to do
registerInterpreterFromPath(interpreterDirString, interpreterJson);
registerInterpreterFromResource(cl, interpreterDirString, interpreterJson);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would cause interpreter to be registered twice which make it difficult to configure the default interpreter setting. I change the method signature to return boolean to indicate whether it is registered successfully.
See my comments.

 * Register interpreter by the following ordering
* 1. Register it from path {ZEPPELIN_HOME}/interpreter/{interpreter_name}/
* interpreter-setting.json
* 2. Register it from interpreter-setting.json in classpath
* {ZEPPELIN_HOME}/interpreter/{interpreter_name}
* 3. Register it by Interpreter.register

@bzz

bzz commented Nov 8, 2016

Copy link
Copy Markdown
Member

on mvn clean package -pl \!zeppelin-distribution I got

Tests run: 11, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.193 sec - in org.apache.zeppelin.search.LuceneSearchTest
Results :
Tests in error:
HeliumApplicationFactoryTest.testUnloadOnInterpreterUnbind:232 » ClassCast jav...
HeliumApplicationFactoryTest.testLoadRunUnloadApplication:148 » ClassCast java...
HeliumApplicationFactoryTest.testUnloadOnInterpreterRestart:299 » ClassCast ja...
HeliumApplicationFactoryTest.testUnloadOnParagraphRemove:193 » ClassCast java....
Tests run: 116, Failures: 0, Errors: 4, Skipped: 0
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Zeppelin ........................................... SUCCESS [ 6.317 s]
[INFO] Zeppelin: Interpreter .............................. SUCCESS [02:35 min]
[INFO] Zeppelin: Zengine .................................. FAILURE [02:48 min]

Am I doing something wrong?

@bzz

bzz commented Nov 23, 2016

Copy link
Copy Markdown
Member

@Leemoonsoo do you think it's worth re-basing on latest master?

Tried mvn clean package -pl \!zeppelin-distribution on this branch again, got:

Results :
Tests in error:
HeliumApplicationFactoryTest.testUnloadOnInterpreterUnbind:232 » ClassCast jav...
HeliumApplicationFactoryTest.testLoadRunUnloadApplication:148 » ClassCast java...
HeliumApplicationFactoryTest.testUnloadOnInterpreterRestart:299 » ClassCast ja...
HeliumApplicationFactoryTest.testUnloadOnParagraphRemove:193 » ClassCast java....
Tests run: 116, Failures: 0, Errors: 4, Skipped: 0
[INFO] ------------------------------------------------------------------------
[INFO] Zeppelin ........................................... SUCCESS [ 8.249 s]
[INFO] Zeppelin: Interpreter .............................. SUCCESS [02:30 min]
[INFO] Zeppelin: Zengine .................................. FAILURE [02:47 min]
[INFO] Zeppelin: Display system apis ...................... SKIPPED

My env

Apache Maven 3.3.3 (7994120775791599e205a5524ec3e0dfe41d4a06; 2015-04-22T20:57:37+09:00)
Maven home: /usr/local/Cellar/maven/3.3.3/libexec
Java version: 1.7.0_45, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.7.0_45.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.10.5", arch: "x86_64", family: "mac"

@bzz

bzz commented Nov 30, 2016

Copy link
Copy Markdown
Member

ping @Leemoonsoo

@bzz

bzz commented Dec 14, 2016

Copy link
Copy Markdown
Member

2 weeks ping @Leemoonsoo

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Leemoonsoo@bzz@zjffdu