[ 
https://issues.apache.org/jira/browse/KAFKA-551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13472470#comment-13472470
 ] 

Jun Rao commented on KAFKA-551:
-------------------------------

Yes, you are right. The code looks good. The confusion for me was that slot is 
an index based off 0 and when using it to set entry size, it actually truncates 
the entry at slot. +1 on the patch.
                
> Log.truncateTo() may need to trucate immutable log segment
> ----------------------------------------------------------
>
>                 Key: KAFKA-551
>                 URL: https://issues.apache.org/jira/browse/KAFKA-551
>             Project: Kafka
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 0.8
>            Reporter: Jun Rao
>            Priority: Blocker
>              Labels: bugs
>         Attachments: KAFKA-551.patch
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> In makeFollower, we need to first truncate the local log to high watermark. 
> It's possible that we need to truncate into segments before the last one. The 
> problem is that all segments except the last one are immutable. So the 
> truncation will fail which prevents the replica fetcher from being started.
> One solution is to reopen the segment as mutable during truncation, if it's 
> not mutable already.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to