Skip to content

[ZEPPELIN-1534] Does not load dependency library when creating new interpreter.#1507

Closed
astroshim wants to merge 1 commit intoapache:masterfrom
astroshim:ZEPPELIN-1534
Closed

[ZEPPELIN-1534] Does not load dependency library when creating new interpreter.#1507
astroshim wants to merge 1 commit intoapache:masterfrom
astroshim:ZEPPELIN-1534

Conversation

@astroshim
Copy link
Copy Markdown
Contributor

What is this PR for?

This PR fixes dependency library loading bug when creating new interpreter.

What type of PR is it?

Bug Fix

What is the Jira issue?

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

How should this be tested?

  1. Create new JDBC interpreter refer to http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/jdbc.html#mysql.
  2. Create new paragraph for testing new JDBC interpreter.
  3. Run paragraph.

Questions:

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

@minahlee
Copy link
Copy Markdown
Member

Tested and LGTM

@asfgit asfgit closed this in abf0470 Oct 19, 2016
darionyaphet pushed a commit to darionyaphet/zeppelin that referenced this pull request Oct 27, 2016
…terpreter.

### What is this PR for?
This PR fixes dependency library loading bug when creating new interpreter.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1534

### How should this be tested?
1. Create new JDBC interpreter refer to http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/jdbc.html#mysql.
2. Create new paragraph for testing new JDBC interpreter.
3. Run paragraph.

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: astroshim <[email protected]>

Closes apache#1507 from astroshim/ZEPPELIN-1534 and squashes the following commits:

aebe352 [astroshim] add loadInterpreterDependencies function in createNewSetting()
pedrozatta pushed a commit to pedrozatta/zeppelin that referenced this pull request Oct 27, 2016
…terpreter.

### What is this PR for?
This PR fixes dependency library loading bug when creating new interpreter.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1534

### How should this be tested?
1. Create new JDBC interpreter refer to http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/jdbc.html#mysql.
2. Create new paragraph for testing new JDBC interpreter.
3. Run paragraph.

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: astroshim <[email protected]>

Closes apache#1507 from astroshim/ZEPPELIN-1534 and squashes the following commits:

aebe352 [astroshim] add loadInterpreterDependencies function in createNewSetting()
Sign up for free to 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.

2 participants