[pulsar] branch master updated: Make bookkeeper shell more userfriendly (#9281)

[mm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new 25dbc94  Make bookkeeper shell more userfriendly (#9281)
25dbc94 is described below

commit 25dbc940a9b79c5c8e8a69201d5bdb972fb6b2a7
Author: Enrico Olivelli <eo...@gmail.com>
AuthorDate: Wed Jan 27 00:55:14 2021 +0100

    Make bookkeeper shell more userfriendly (#9281)
    
    Co-authored-by: Enrico Olivelli <eo...@apache.org>
---
 bin/bookkeeper | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/bin/bookkeeper b/bin/bookkeeper
index 57435d9..93ef29b 100755
--- a/bin/bookkeeper
+++ b/bin/bookkeeper
@@ -38,7 +38,6 @@ fi
 
 if [ "x$JMXDISABLE" = "x" ]
 then
-    echo "JMX enabled by default" >&2
     # for some reason these two options are necessary on jdk6 on Ubuntu
     #   accord to the docs they are not necessary, but otw jconsole cannot
     #   do a local attach
@@ -170,6 +169,12 @@ OPTS="$OPTS -Djava.net.preferIPv4Stack=true"
 
 # log directory & file
 BOOKIE_LOG_APPENDER=${BOOKIE_LOG_APPENDER:-"Console"}
+
+# send bookie shell output to console unconditionally
+if [ $COMMAND == "shell" ]; then
+BOOKIE_LOG_APPENDER=Console
+fi
+
 BOOKIE_LOG_DIR=${BOOKIE_LOG_DIR:-"$BK_HOME/logs"}
 BOOKIE_LOG_FILE=${BOOKIE_LOG_FILE:-"bookkeeper-server.log"}