You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "Lianet Magrans (Jira)" <ji...@apache.org> on 2024/09/10 11:11:00 UTC

[jira] [Updated] (KAFKA-16966) Allow offset commit fetch to reuse previous request if partitions are a subset

     [ https://issues.apache.org/jira/browse/KAFKA-16966?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lianet Magrans updated KAFKA-16966:
-----------------------------------
    Priority: Minor  (was: Major)

> Allow offset commit fetch to reuse previous request if partitions are a subset
> ------------------------------------------------------------------------------
>
>                 Key: KAFKA-16966
>                 URL: https://issues.apache.org/jira/browse/KAFKA-16966
>             Project: Kafka
>          Issue Type: Improvement
>          Components: clients, consumer
>    Affects Versions: 3.8.0
>            Reporter: Kirk True
>            Priority: Minor
>              Labels: consumer-threading-refactor
>             Fix For: 4.0.0
>
>
> In {{{}initWithCommittedOffsetsIfNeeded{}}}, the behavior of the existing {{LegacyKafkaConsumer}} is to allow reuse only if the partitions for the _current_ request equal those of the _previous_ request *exactly* ([source|https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/clients/consumer/internals/ConsumerCoordinator.java?rgh-link-date=2024-06-14T16%3A43%3A11Z#L927]). That logic is the basis for the behavior used in the {{{}AsyncKafkaConsumer{}}}.
> The proposed change is to allow for request reuse if the partitions for the _current_ request are a subset of those of the _previous_ request. This introduces a subtle difference in behavior between the two {{Consumer}} implementations, so we need to decided if we want to change both implementations or just {{{}AsyncKafkaConsumer{}}}. Also, the specific case that the request reuse logic solves is when the user has passed in a very low timeout value in a tight {{poll()}} loop, which suggests the partitions wouldn't be changing between those loops.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)