zjffdu opened a new pull request #4208:
URL: https://github.com/apache/zeppelin/pull/4208
### What is this PR for?
Should close `clientPool` explicitly, otherwise it won't be garbage
collected.
### What type of PR is it?
[Bug Fix]
### Todos
* [ ] - Task
##
Reamer commented on pull request #4208:
URL: https://github.com/apache/zeppelin/pull/4208#issuecomment-905393394
We should implement the [`AutoCloseable`
interface](https://docs.oracle.com/javase/8/docs/api/java/lang/AutoCloseable.html)
in this class.
Maybe we can use the `try-with-reso
Reamer commented on a change in pull request #4207:
URL: https://github.com/apache/zeppelin/pull/4207#discussion_r695852869
##
File path: shell/src/main/java/org/apache/zeppelin/shell/ShellInterpreter.java
##
@@ -213,7 +214,17 @@ public int getProgress(InterpreterContext contex
rickchengx commented on a change in pull request #4174:
URL: https://github.com/apache/zeppelin/pull/4174#discussion_r696215340
##
File path:
zeppelin-plugins/launcher/k8s-standard/src/main/java/org/apache/zeppelin/interpreter/launcher/K8sRemoteInterpreterProcess.java
##
@@ -2
rickchengx commented on pull request #4174:
URL: https://github.com/apache/zeppelin/pull/4174#issuecomment-906001473
> > Now by default, user can create the interpreter pod in the `default`
namespace.
>
> Is it necessary to add a namespace?
Now the `zeppelin-server.yaml` has a
zjffdu commented on pull request #4208:
URL: https://github.com/apache/zeppelin/pull/4208#issuecomment-906021293
@Reamer I am afraid we can not use `try-with-resource` here, because
`PooledRemoteClient` will be used in the whole lifecycle of
`RemoteInterpreterProcess` (used in multiple pl
