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 interpreter related files together may make
interpreter image too large.
2. it’s not flexible to change the interpreter related files,for example, if
we need to change the flink 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]