You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@kafka.apache.org by ijuma <gi...@git.apache.org> on 2017/10/04 14:25:50 UTC

[GitHub] kafka pull request #4016: MINOR: Simplify log cleaner and fix compiler warni...

GitHub user ijuma opened a pull request:

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

    MINOR: Simplify log cleaner and fix compiler warnings

    - Simplify LogCleaner.cleanSegments and add comment regarding thread
    unsafe usage of `LogSegment.append`. This was a result of investigating
    KAFKA-4972.
    - Fix compiler warnings.

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

    $ git pull https://github.com/ijuma/kafka simplify-log-cleaner-and-fix-warnings

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

    https://github.com/apache/kafka/pull/4016.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 #4016
    
----
commit 3b26b21c4a41b9857d48a09a63a560228924df4f
Author: Ismael Juma <is...@juma.me.uk>
Date:   2017-10-04T13:57:03Z

    Simplify LogCleaner.cleanSegments and add comment regarding thread unsafe usage of `LogSegment.append`

commit a1e50d8fbffc977646397f0446efeaa798816d87
Author: Ismael Juma <is...@juma.me.uk>
Date:   2017-10-04T13:57:20Z

    Fix compiler warnings

----


---

[GitHub] kafka pull request #4016: MINOR: Simplify log cleaner and fix compiler warni...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---