You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by zj...@apache.org on 2019/10/14 01:55:33 UTC

[zeppelin] branch master updated: [HOTFIX] Code format fixing

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 a5aebc9  [HOTFIX] Code format fixing
a5aebc9 is described below

commit a5aebc9908529efc6340208d368f7d65bc87cd19
Author: Jeff Zhang <zj...@apache.org>
AuthorDate: Mon Oct 14 09:55:21 2019 +0800

    [HOTFIX] Code format fixing
---
 .../apache/zeppelin/interpreter/launcher/DockerInterpreterProcess.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/zeppelin-plugins/launcher/docker/src/main/java/org/apache/zeppelin/interpreter/launcher/DockerInterpreterProcess.java b/zeppelin-plugins/launcher/docker/src/main/java/org/apache/zeppelin/interpreter/launcher/DockerInterpreterProcess.java
index 779c42b..f3139c6 100644
--- a/zeppelin-plugins/launcher/docker/src/main/java/org/apache/zeppelin/interpreter/launcher/DockerInterpreterProcess.java
+++ b/zeppelin-plugins/launcher/docker/src/main/java/org/apache/zeppelin/interpreter/launcher/DockerInterpreterProcess.java
@@ -454,7 +454,8 @@ public class DockerInterpreterProcess extends RemoteInterpreterProcess {
       LOGGER.warn("{} file not found, Did not upload the krb5.conf to the container!", krb5conf);
     }
 
-    // TODO: Interpreter specific settings, we should consider general property or some other more elegant solution
+    // TODO(Vince): Interpreter specific settings, we should consider general property or some
+    // other more elegant solution
     // 3) Get the keytab file in each interpreter properties
     // Upload Keytab file to container, Keep the same directory as local host
     // 3.1) shell interpreter properties keytab file