You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2021/05/03 10:58:32 UTC

[GitHub] [zeppelin] cuspymd commented on a change in pull request #4104: [ZEPPELIN-5338]. Add logging prefix in interpreter.sh for easy diagnose

cuspymd commented on a change in pull request #4104:
URL: https://github.com/apache/zeppelin/pull/4104#discussion_r625012234



##########
File path: bin/interpreter.sh
##########
@@ -296,5 +296,6 @@ else
 fi
 
 # Don't remove this echo, it is for diagnose, this line of output will be redirected to java log4j output.
-echo "Interpreter launch command: ${INTERPRETER_RUN_COMMAND[@]}"
+# Output that starts with `[INFO]` will be redirected to log4j output.

Review comment:
       Isn't it better to express "log4j INFO output" or "log4j INFO message" more precisely than "log4j output"? Depending on the log level setting, debug messages can also be displayed in the output.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org