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

Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/10/23 11:06:58 UTC

[GitHub] [incubator-kvrocks] git-hulk commented on issue #662: Consider only output to LOG instead of hardcoded stdout/stderr

git-hulk commented on issue #662:
URL: https://github.com/apache/incubator-kvrocks/issues/662#issuecomment-1288087609

   > cc @git-hulk
   > 
   > <img alt="image" width="1728" src="https://user-images.githubusercontent.com/18818196/197383662-02ed392d-69be-447a-82e6-f2b0c0cd6870.png">
   > 
   > Currently, I found only these three lines in the source code that we print to the stdout in starting up logic. I don't know whether we should have a rule that only output to LOG everywhere and the current status looks good to me.
   
   To be honest, we don't have an explicit rule on when should send logs to the stdout instead of LOG file, but I think all logs should go into LOG except:
   - command outputs like `./kvrock --help`
   - logs before the logging instance was setup


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

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

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