Skip to content

[ZEPPELIN-914] Apply new mechanism to IgniteInterpreter - #1819

Closed
DrIgor wants to merge 3 commits into
apache:masterfrom
DrIgor:ZEPPELIN-914
Closed

[ZEPPELIN-914] Apply new mechanism to IgniteInterpreter#1819
DrIgor wants to merge 3 commits into
apache:masterfrom
DrIgor:ZEPPELIN-914

Conversation

@DrIgor

@DrIgorDrIgor commented Dec 29, 2016

Copy link
Copy Markdown
Contributor

What is this PR for?

This handles replacing the registration of interpreter with static block by the interpreter-setting.json file

What type of PR is it?

[Improvement]

What is the Jira issue?

ZEPPELIN-914

How should this be tested?

There shouldn't be any warning like below on starting the server

INFO 2016-09-29 00:25:46,247 - Bootstrapping Ignite Interpreter
WARN 2016-09-29 00:25:46,250 - Static initialization is deprecated for interpreter Ignite, You should change it to use interpreter-setting.json in your jar or interpreter/{interpreter}/interpreter-setting.json
INFO 2016-09-29 00:25:46,250 - Inclass=org.apache.zeppelin.ignite

And ensure that the Ignite related paragraphs run without any error

Questions:

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

@DrIgor

Copy link
Copy Markdown
ContributorAuthor

Original pull request #1561

IgniteSqlInterpreterTest is known flaky test: ZEPPELIN-1738

@DrIgor

Copy link
Copy Markdown
ContributorAuthor

Travis fails with:

testBasicIntp(org.apache.zeppelin.spark.PySparkInterpreterTest) Time elapsed: 58.947 sec <<< FAILURE!
java.lang.AssertionError: expected:<SUCCESS> but was:<ERROR>

Known issue: ZEPPELIN-1746

@jongyoul

Copy link
Copy Markdown
Member

Thanks!! LGTM. Will merge it

@DrIgor
DrIgor deleted the ZEPPELIN-914 branch January 13, 2017 06:08
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

@DrIgor@jongyoul@meenakshisekar