[GitHub] [bookkeeper] eolivelli commented on pull request #2455: BP-40: Convert all System.*.print statements to LOG.* statements

[GitBox <gi...@apache.org>]

eolivelli commented on pull request #2455:
URL: https://github.com/apache/bookkeeper/pull/2455#issuecomment-716496303


   I am not sure this is good.
   I have to think about it a bit more.
   How can we distinguish the output for the user from the logs of the application ?
   
   The logger usually is something that writes to a file useful information, it is not part of the UI (the stuff the user sees)
   
   for instance, if I want to ask for a username I am not going to use
   log.info("Please tell me the username:")
   console.readLine();
   
   but
   console.println("Please tell me the username:")
   console.readLine();
   
   these tool are interactive, I am not sure the log system is what we want to use


----------------------------------------------------------------
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