yifan-c commented on code in PR #58:
URL: https://github.com/apache/cassandra-sidecar/pull/58#discussion_r1310545898
##########
src/test/resources/logback-test.xml:
##########
@@ -20,7 +20,7 @@
<appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
<filter class="ch.qos.logback.classic.filter.ThresholdFilter">
- <level>INFO</level>
+ <level>DEBUG</level>
Review Comment:
Typically `Debug` is used when running test. What are the other scenarios of
using debug? It is (almost) never considered in production. So.. if we do not
really enable debug level, what is the point of having those debug message in
the code :D
I think the way too many log messages is a different issue. Some useless
chatty logger can be disabled or set to INFO.
We need to put more thoughts. For now, I am fine with "INFO" if Francisco
feels strong about it.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
