yaooqinn commented on a change in pull request #31804:
URL: https://github.com/apache/spark/pull/31804#discussion_r593753415
##########
File path: sql/core/src/test/resources/log4j.properties
##########
@@ -22,7 +22,7 @@ log4j.rootLogger=INFO, CA, FA
log4j.appender.CA=org.apache.log4j.ConsoleAppender
log4j.appender.CA.layout=org.apache.log4j.PatternLayout
log4j.appender.CA.layout.ConversionPattern=%d{HH:mm:ss.SSS} %p %c: %m%n
-log4j.appender.CA.Threshold = WARN
+log4j.appender.CA.Threshold = FATAL
Review comment:
yeah, I guess so. we can mute console output for most modules. we call
`o.s.Assertions.intercept` frequently which produces a lot of unnecessary error
logs.
When we mute them, the error stacktraces for failed tests can still be kept.
I can make a separate PR to fix it if it makes sense to the CCers.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
