You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by mo...@apache.org on 2016/09/01 20:14:44 UTC
zeppelin git commit: [MINOR] Add new line before logging paragraph
content
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 <lr...@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 <lr...@apache.org>
Authored: Sun Aug 28 16:49:02 2016 -0700
Committer: Lee moon soo <mo...@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);