[ZOOKEEPER-2393] Revert run-time dependency on log4j and slf4j-log4j12

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 3.5.2, 3.6.0
• Component/s: server
• Hadoop Flags: Reviewed
• Assignee: Mohammad Arshad
• Reporter: Mohammad Arshad
• Created: 18/Mar/16 01:39
• Updated: 21/Jul/16 20:18
• Resolved: 19/Mar/16 21:56

Description

Zookeeper run-time dependency on log4j and slf4j-log4j12 was removed as part of ZOOKEEPER-1371 jira work.
Following things were done as part of ZOOKEEPER-1371

1. Removed direct log4j API use from the code, instead used slf4j-api
2. Changed log4j and slf4j-log4j12 run time dependency to test time dependency
3. Upgraded log4j, slf4j-log4j12 and slf4j-api versions.
   Here is the component wise version change
   • (zookeeper)ivy.xml
     log4j: 1.2.15 -->1.7.5
   • src\contrib\loggraph\ivy.xml
     slf4j-api: 1.6.1 -->1.7.5
     slf4j-log4j12: 1.6.1 -->1.7.5
     log4j: 1.2.15 -->1.7.5
   • src\contrib\rest\ivy.xml
     slf4j-api: 1.6.1 -->1.7.5
     slf4j-log4j12: 1.6.1 -->1.7.5
     log4j: 1.2.15 -->1.7.5
   • src\contrib\zooinspector\ivy.xml
     slf4j-api: 1.6.1 -->1.7.5
     slf4j-log4j12: 1.6.1 -->1.7.5
     log4j: 1.2.15 -->1.7.5

The major problem with ZOOKEEPER-1371 change is that it removed run time dependency. For more detail refer ZOOKEEPER-2342 discussion
Now as part of this jira revert back only run time dependency, #2, on log4j and slf4j-log4j12.

Issue Links

• is related to: ZOOKEEPER-2342 Migrate to Log4J 2.
• is related to: ZOOKEEPER-1371 Remove dependency on log4j in the source code.