redapel commented on a change in pull request #564:
URL: https://github.com/apache/logging-log4j2/pull/564#discussion_r697825524
##########
File path: log4j-mongodb4/pom.xml
##########
@@ -88,6 +88,17 @@
<artifactId>de.flapdoodle.embed.mongo</artifactId>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.apache.logging.log4j</groupId>
+ <artifactId>log4j-slf4j-impl</artifactId>
Review comment:
The test is about using slf4j on top of log4j2 with mongodb4 appender.
By using slf4j without adding that dependency, the log message (Hello log) will
not get written to the mongodb database so the test would fail. Also there are
warning message being shown :
`SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".`
`SLF4J: Defaulting to no-operation (NOP) logger implementation`
`SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further
details.`
Do i need to remove that added dependency or something?
--
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]
