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

chia7712 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 5c49b48eb07 MINOR: Correct javadoc and comment in 
AbstractFetch#fetchablePartitions (#20790)
5c49b48eb07 is described below

commit 5c49b48eb0732043400ec1ffb23022ebb3b47085
Author: majialong <[email protected]>
AuthorDate: Thu Oct 30 10:18:12 2025 +0800

    MINOR: Correct javadoc and comment in AbstractFetch#fetchablePartitions 
(#20790)
    
    This PR is a minor improvement on the [previous
    PR](https://github.com/apache/kafka/pull/20745), correcting the javadoc
    and comment information.
    
    Reviewers: TaiJuWu <[email protected]>, Chia-Ping Tsai
     <[email protected]>
---
 .../org/apache/kafka/clients/consumer/internals/AbstractFetch.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractFetch.java
 
b/clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractFetch.java
index 27f7fce1fab..c20c5bd8cfb 100644
--- 
a/clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractFetch.java
+++ 
b/clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractFetch.java
@@ -335,7 +335,7 @@ public abstract class AbstractFetch implements Closeable {
     }
 
     /**
-     * Return the set of <em>fetchable</em> partitions, which are the set of 
partitions to which we are subscribed,
+     * Return the list of <em>fetchable</em> partitions, which are the list of 
partitions to which we are subscribed,
      * but <em>excluding</em> any partitions for which we still have buffered 
data. The idea is that since the user
      * has yet to process the data for the partition that has already been 
fetched, we should not go send for more data
      * until the previously-fetched data has been processed.
@@ -429,7 +429,7 @@ public abstract class AbstractFetch implements Closeable {
         // This is the set of partitions that have buffered data
         Set<TopicPartition> buffered = 
Collections.unmodifiableSet(fetchBuffer.bufferedPartitions());
 
-        // This is the set of partitions that do not have buffered data
+        // This is the list of partitions that are fetchable and have no 
buffered data
         List<TopicPartition> unbuffered = fetchablePartitions(buffered);
 
         if (unbuffered.isEmpty()) {

Reply via email to