[zeppelin] branch master updated: [ZEPPELIN-4564] Changed path separator in Python interpreter to allow it to work on windows as well

[zjffdu]

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

zjffdu 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 586e823  [ZEPPELIN-4564] Changed path separator in Python interpreter 
to allow it to work on windows as well
586e823 is described below

commit 586e823e1630e054a6260b490a3322fcee366017
Author: Muhammad Taufiq <muhammad.tau...@morganstanley.com>
AuthorDate: Thu Jan 16 14:11:25 2020 +0000

    [ZEPPELIN-4564] Changed path separator in Python interpreter to allow it to 
work on windows as well
    
    ### What is this PR for?
    Python Interpreter fails on Windows because of classpath issue
    
    ### What type of PR is it?
    Bug Fix
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-4564#
    
    THIS SOFTWARE IS CONTRIBUTED SUBJECT TO THE TERMS OF THE APACHE SOFTWARE 
FOUNDATION SOFTWARE GRANT AND CORPORATE CONTRIBUTOR LICENSE AGREEMENT VERSION 
R190612.
    THIS SOFTWARE IS LICENSED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" 
AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE 
IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE AND 
ANY WARRANTY OF NON-INFRINGEMENT, ARE DISCLAIMED. IN NO EVENT SHALL THE 
COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 
SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, 
PROCUREMENT OF SUBSTITUTE GOODS [...]
    
    Author: Muhammad Taufiq <muhammad.tau...@morganstanley.com>
    
    Closes #3604 from Muhammad-ms/zeppelin9_file_separator and squashes the 
following commits:
    
    a09b6463a [Muhammad Taufiq] Changed path separator to allow it to work on 
windows as well
---
 python/src/main/java/org/apache/zeppelin/python/PythonInterpreter.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/python/src/main/java/org/apache/zeppelin/python/PythonInterpreter.java 
b/python/src/main/java/org/apache/zeppelin/python/PythonInterpreter.java
index 4d8a4b7..e4ea732 100644
--- a/python/src/main/java/org/apache/zeppelin/python/PythonInterpreter.java
+++ b/python/src/main/java/org/apache/zeppelin/python/PythonInterpreter.java
@@ -219,7 +219,7 @@ public class PythonInterpreter extends Interpreter {
     if (!env.containsKey("PYTHONPATH")) {
       env.put("PYTHONPATH", path);
     } else {
-      env.put("PYTHONPATH", env.get("PYTHONPATH") + ":" + path);
+      env.put("PYTHONPATH", env.get("PYTHONPATH") + File.pathSeparator + path);
     }
   }