Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-2393

Revert run-time dependency on log4j and slf4j-log4j12

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • None
    • 3.5.2, 3.6.0
    • server
    • None
    • Reviewed

    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.

      Attachments

        1. ZOOKEEPER-2393.patch
          31 kB
          Mohammad Arshad
        2. ZOOKEEPER-2393.patch
          33 kB
          Mohammad Arshad

        Issue Links

          Activity

            People

              arshad.mohammad Mohammad Arshad
              arshad.mohammad Mohammad Arshad
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: