You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by sh...@apache.org on 2022/04/15 13:00:37 UTC

[kafka] branch trunk updated: MINOR: update comment in LocalLog.replaceSegments() (#12054)

This is an automated email from the ASF dual-hosted git repository.

showuon pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/trunk by this push:
     new f4e3ccd81a MINOR: update comment in LocalLog.replaceSegments() (#12054)
f4e3ccd81a is described below

commit f4e3ccd81af453dae7cf0ec5b383882528a3a91f
Author: Xiaobing Fang <bi...@qq.com>
AuthorDate: Fri Apr 15 21:00:11 2022 +0800

    MINOR: update comment in LocalLog.replaceSegments() (#12054)
    
    Reviewers: Luke Chen <sh...@gmail.com>
---
 core/src/main/scala/kafka/log/LocalLog.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/scala/kafka/log/LocalLog.scala b/core/src/main/scala/kafka/log/LocalLog.scala
index 8e83070ff0..620f33119a 100644
--- a/core/src/main/scala/kafka/log/LocalLog.scala
+++ b/core/src/main/scala/kafka/log/LocalLog.scala
@@ -923,7 +923,7 @@ object LocalLog extends Logging {
                                    isRecoveredSwapFile: Boolean = false): Iterable[LogSegment] = {
     val sortedNewSegments = newSegments.sortBy(_.baseOffset)
     // Some old segments may have been removed from index and scheduled for async deletion after the caller reads segments
-    // but before this method is executed. We want to filter out those segments to avoid calling asyncDeleteSegment()
+    // but before this method is executed. We want to filter out those segments to avoid calling deleteSegmentFiles()
     // multiple times for the same segment.
     val sortedOldSegments = oldSegments.filter(seg => existingSegments.contains(seg.baseOffset)).sortBy(_.baseOffset)