[
https://issues.apache.org/jira/browse/KAFKA-6568?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16369584#comment-16369584
]
ASF GitHub Bot commented on KAFKA-6568:
---------------------------------------
becketqin opened a new pull request #4592: MINOR follow-up for KAFKA-6568
URL: https://github.com/apache/kafka/pull/4592
@ijuma This is the minor follow-up patch for #4580 to address your comments.
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage and CI build status
- [ ] Verify documentation (including upgrade notes)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> LogCleanerManager.doneDeleting() should check the partition state before
> deleting the in progress partition
> -----------------------------------------------------------------------------------------------------------
>
> Key: KAFKA-6568
> URL: https://issues.apache.org/jira/browse/KAFKA-6568
> Project: Kafka
> Issue Type: Bug
> Affects Versions: 0.10.1.1, 0.10.2.1, 1.0.0, 0.11.0.2
> Reporter: Jiangjie Qin
> Assignee: Jiangjie Qin
> Priority: Blocker
> Fix For: 1.1.0, 0.11.0.3, 1.0.2
>
>
> {{LogCleanerManager.doneDeleting()}} removes the partition from the
> {{inProgress}} map without checking if the partition is paused or not. This
> will cause the paused partition state to be lost, and may also cause another
> thread calling {{LogCleanerManager.abortAndPauseCleaning()}} to block
> indefinitely waiting on the partition state to become paused.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
