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 <jongyoul@gmail.com>
(cherry picked from commit 334da53)
Signed-off-by: Jongyoul Lee <jongyoul@gmail.com>
  • Loading branch information
Reamer authored and jongyoul committed Oct 1, 2024
1 parent 88bccf9 commit 82e85ae
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 82e85ae

Please sign in to comment.