Github user anmolnar commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/451#discussion_r166669488
--- Diff: src/java/test/org/apache/zookeeper/test/ReadOnlyModeTest.java ---
@@ -239,13 +243,13 @@ public void testSessionEstablishment() throws
Exception {
public void testSeekForRwServer() throws Exception {
// setup the logger to capture all logs
- Layout layout = Logger.getRootLogger().getAppender("CONSOLE")
+ Layout layout =
org.apache.log4j.Logger.getRootLogger().getAppender("CONSOLE")
--- End diff --
slf4j's string formatter feature is being used in these tests. I wouldn't
refactor them to be honest.
---
