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 <lrese...@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 <lrese...@apache.org> Authored: Sun Aug 28 16:49:02 2016 -0700 Committer: Lee moon soo <m...@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);
