zeppelin-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From m...@apache.org
Subject zeppelin git commit: [MINOR] Add new line before logging paragraph content
Date Thu, 01 Sep 2016 20:14:44 GMT
Repository: zeppelin
Updated Branches:
  refs/heads/master 323aa1883 -> da8857fea


[MINOR] Add new line before logging paragraph content

### What is this PR for?
Add new line before logging paragraph content

### What type of PR is it?
[Improvement]

Author: Luciano Resende <lresende@apache.org>

Closes #1374 from lresende/debug and squashes the following commits:

0f6e80c [Luciano Resende] [MINOR] Add new line before logging paragraph content


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/da8857fe
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/da8857fe
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/da8857fe

Branch: refs/heads/master
Commit: da8857fea8edd995a8c958c77c33f411add9a660
Parents: 323aa18
Author: Luciano Resende <lresende@apache.org>
Authored: Sun Aug 28 16:49:02 2016 -0700
Committer: Lee moon soo <moon@apache.org>
Committed: Fri Sep 2 06:14:35 2016 +1000

----------------------------------------------------------------------
 .../apache/zeppelin/interpreter/remote/RemoteInterpreter.java    | 4 +++-
 .../zeppelin/interpreter/remote/RemoteInterpreterServer.java     | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/da8857fe/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java
----------------------------------------------------------------------
diff --git a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java
b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java
index db740f4..3e6fa39 100644
--- a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java
+++ b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java
@@ -295,7 +295,9 @@ public class RemoteInterpreter extends Interpreter {
 
   @Override
   public InterpreterResult interpret(String st, InterpreterContext context) {
-    logger.debug("st: {}", st);
+    if (logger.isDebugEnabled()) {
+      logger.debug("st:\n{}", st);
+    }
     FormType form = getFormType();
     RemoteInterpreterProcess interpreterProcess = getInterpreterProcess();
     Client client = null;

http://git-wip-us.apache.org/repos/asf/zeppelin/blob/da8857fe/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
----------------------------------------------------------------------
diff --git a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
index 57dd768..7ddb928 100644
--- a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
+++ b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
@@ -285,7 +285,9 @@ public class RemoteInterpreterServer
   @Override
   public RemoteInterpreterResult interpret(String noteId, String className, String st,
       RemoteInterpreterContext interpreterContext) throws TException {
-    logger.debug("st: {}", st);
+    if (logger.isDebugEnabled()) {
+      logger.debug("st:\n{}", st);
+    }
     Interpreter intp = getInterpreter(noteId, className);
     InterpreterContext context = convert(interpreterContext);
 


Mime
View raw message