This is an automated email from the ASF dual-hosted git repository.

jongyoul pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/master by this push:
     new 334da53d7f [ZEPPELIN-6103] Revert "[HOTFIX] Escape envs when using 
`.conf` (#4715)"
334da53d7f is described below

commit 334da53d7f2f3f8fe13128a6ec944853e9216452
Author: Philipp Dallig <philipp.dal...@gmail.com>
AuthorDate: Tue Oct 1 14:09:57 2024 +0200

    [ZEPPELIN-6103] Revert "[HOTFIX] Escape envs when using `.conf` (#4715)"
    
    ### What is this PR for?
    This PR reverts commit dd08a3966ef3b0b40f13d0291d7cac5ec3dd9f9c 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 <jongy...@gmail.com>
---
 bin/interpreter.sh | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/bin/interpreter.sh b/bin/interpreter.sh
index cf8501bb3c..19a6fe7119 100755
--- a/bin/interpreter.sh
+++ b/bin/interpreter.sh
@@ -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')

Reply via email to