Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/2106
  
    Tested `%python.conda` and `%python.pandasql` and it works well.
    Could you restore PythonDockerInterpreter.java and 
PythonDockerInterpreterTest.java for `%python.docker` as well?
    
    To make it work, i think we need to let docker container mount directory 
that py4j exists using -v option, and then set PYTHONPATH env variable to load 
py4j library using -e option.
    
    So this line
    ```java
    setPythonCommand("docker run -i --rm " + image + " python -iu");
    ```
    
    can be updated to something like
    
    ```java
    String py4jDir = ZEPPELIN_HOME + "/interpreter/python";
    String py4jFile = "py4j-{version}.zip";   // read directory py4jDir and get 
file starts with py4j
    setPythonCommand("docker run -i --rm -v " + py4jDir + ":/zeppelin_python -e 
/zeppelin_python/" + py4jFile + " " + image + " python -iu");
    ```



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to