Skip to content

Commit

Permalink
[ZEPPELIN-6103] Revert "[HOTFIX] Escape envs when using .conf (#4715)"
Browse files Browse the repository at this point in the history
### What is this PR for?
This PR reverts commit dd08a39 which was merged with #4715

I have set the environment variable `ZEPPELIN_INTP_CLASSPATH_OVERRIDES`, which now no longer works due to the change.

I have set the environment variable to the value `/usr/share/java/*`, which in my opinion represents a valid classpath.

`printf %q "/usr/share/java/*"` results in `/usr/share/java/\*`, which is no longer a valid classpath.

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

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

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


Closes #4843 from Reamer/revert_env_hotfix.

Signed-off-by: Jongyoul Lee <[email protected]>
  • Loading branch information
Reamer authored Oct 1, 2024
1 parent 1633c52 commit 334da53
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions bin/interpreter.sh
Original file line number Diff line number Diff line change
Expand Up @@ -101,9 +101,6 @@ fi

. "${bin}/common.sh"

# Escape envs
ZEPPELIN_INTP_CLASSPATH_OVERRIDES=$(printf %q "${ZEPPELIN_INTP_CLASSPATH_OVERRIDES}")

check_java_version

ZEPPELIN_INTERPRETER_API_JAR=$(find "${ZEPPELIN_HOME}/interpreter" -name 'zeppelin-interpreter-shaded-*.jar')
Expand Down

0 comments on commit 334da53

Please sign in to comment.