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:
[email protected]
