[ZOOKEEPER-850] Switch from log4j to slf4j

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.3.1
• Fix Version/s: 3.4.0
• Component/s: java client
• Hadoop Flags: Reviewed

Release Note

* replaces log4j with slf4j code (also in contrib for bookkeeper, zooinspector,rest,loggraph), added slf4j dependencies into several ivy.xml files
* you must add slf4j-api-1.6.1.jar and slf4j-log4j12-1.6.1.jar (bridge from sl4j to log4j) to the classpath, if not using the standard scripts
* log4j remains as the final logger yet, there is still work to do: remove programmatic access to the log4j api from certain classes (which add appenders or configure log4j at runtime), or move them to contrib

• Assignee: Olaf Krische
• Reporter: Olaf Krische
• Created: 16/Aug/10 12:49
• Updated: 25/Feb/20 01:21
• Resolved: 24/Jan/12 00:25

Description

Hello,

i would like to see slf4j integrated into the zookeeper instead of relying explicitly on log4j.

slf4j is an abstract logging framework. There are adapters from slf4j to many logger implementations, one of them is log4j.

The decision which log engine to use i dont like to make so early.

This would help me to embed zookeeper in my own applications (which use a different logger implemenation, but slf4j is the basis)

What do you think?

(as i can see, those slf4j request flood all other projects on apache as well

Maybe for 3.4 or 4.0?

I can offer a patchset, i have experience in such an migration already.

Issue Links

• blocks: SOLR-2369 Zookeeper depends on log4j, thus also SolrCloud does
• relates to: ZOOKEEPER-1371 Remove dependency on log4j in the source code.
• relates to: ZOOKEEPER-3737 Unable to eliminate log4j1 transitive dependency

Sub-Tasks

• Remove or move ManagedUtil to contrib, because it has direct log4j dependencies: Resolved