zhoulii commented on a change in pull request #3884:
URL: https://github.com/apache/zeppelin/pull/3884#discussion_r475059807



##########
File path: k8s/interpreter/100-interpreter-spec.yaml
##########
@@ -78,6 +78,21 @@ spec:
   - name: spark-home
     emptyDir: {}
   {% endif %}
+  {% if zeppelin.k8s.interpreter.group.name == "flink" %}
+    volumeMounts:
+    - name: flink-home
+      mountPath: /flink
+  initContainers:
+  - name: flink-home-init
+    image: {{zeppelin.k8s.flink.container.image}}
+    command: ["sh", "-c", "cp -r /opt/flink/* /flink/"]

Review comment:
       Hi @Reamer , the way that add Python and R packages is ok,but it may 
have some limitations, for example:
   
   1. add different kinds of related packages together may make interpreter 
image too large.
   2. it’s not flexible to change the package,for example, if we need to change 
the package version, we have to rebuild interpreter image.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to