Skip to content

ZEPPELIN-383 Override classpath with ZEPPELIN_CLASSPATH_OVERRIDES - #412

Closed
echarles wants to merge 2 commits into
apache:masterfrom
echarles:ZEPPELIN-383-CLASSPATH_OVERRIDES
Closed

ZEPPELIN-383 Override classpath with ZEPPELIN_CLASSPATH_OVERRIDES#412
echarles wants to merge 2 commits into
apache:masterfrom
echarles:ZEPPELIN-383-CLASSPATH_OVERRIDES

Conversation

@echarles

Copy link
Copy Markdown
Member

Prepend ZEPPELIN_CLASSPATH_OVERRIDES environment variable when building CLASSPATH in the shell scripts.

This PR replaces the closed#398 and #386 - Sorry for the mess...

@Leemoonsoo

Copy link
Copy Markdown
Member

LGTM

@Leemoonsoo

Copy link
Copy Markdown
Member

merge if there're no more discussions

@felixcheung

Copy link
Copy Markdown
Member

looks good

Comment threadbin/interpreter.sh

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@echarles Variable name ends with double S. Maybe its typo?

asfgit pushed a commit that referenced this pull request Nov 13, 2015
#412
fix :: bin/interpreter.sh
ZEPPELIN_CLASSPATH_OVERRIDE**SS** -> ZEPPELIN_CLASSPATH_OVERRIDE**S**
Author: Chae-Sung Lim <estail7s@gmail.com>
Closes#422 from cloverhearts/ZEPPELIN-383-Additional_modifications and squashes the following commits:
90a267e [Chae-Sung Lim] ZEPPELIN-383-Additional_modifications
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.

4 participants

@echarles@Leemoonsoo@felixcheung@minahlee