[GitHub] kafka pull request #4297: KAFKA-6317: Maven artifact for kafka should not de...

[ijuma <gi...@git.apache.org>]

GitHub user ijuma opened a pull request:

    https://github.com/apache/kafka/pull/4297

    KAFKA-6317: Maven artifact for kafka should not depend on log4j

    It should only depend on slf4j-api (like kafka-clients). The
    release tarball still includes log4j and slf4j-log4j12.
    
    Manually verified that there are no duplicate dependencies
    in the release tarball and `./gradlew core:dependencies`
    looks good.
    
    ### Committer Checklist (excluded from commit message)
    - [ ] Verify design and implementation 
    - [ ] Verify test coverage and CI build status
    - [ ] Verify documentation (including upgrade notes)


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ijuma/kafka kafka-6317-kafka-slf4j-api-only

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/4297.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #4297
    
----
commit 544195cf3f1af84c32da9ecc1fe7bd6d2cb83748
Author: Ismael Juma <is...@juma.me.uk>
Date:   2017-12-06T16:58:29Z

    KAFKA-6317: Maven artifact for kafka should not depend on log4j

----


---

[GitHub] kafka pull request #4297: KAFKA-6317: Maven artifact for kafka should not de...

[asfgit <gi...@git.apache.org>]

Github user asfgit closed the pull request at:

    https://github.com/apache/kafka/pull/4297


---