[
https://issues.apache.org/jira/browse/KAFKA-16792?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17878298#comment-17878298
]
Chia-Ping Tsai commented on KAFKA-16792:
----------------------------------------
{quote}
With that we would guarantee that poll(lowTimeout) generates the requests it
needs (just like the classic consumer does), even though it may not have the
time to wait for them.
{quote}
Aligning the poll(0)/close(0) with classic consumer may reduce the panic of
migration. However, waiting for 1 full cycle seems to couple the app-thread to
background-thread. I feel that is not the purpose of `Async`Consumer.
Personally, honoring the timeout passed by users is a good pattern to me.
> Enable consumer unit tests that fail to fetch offsets only for new consumer
> with poll(0)
> ----------------------------------------------------------------------------------------
>
> Key: KAFKA-16792
> URL: https://issues.apache.org/jira/browse/KAFKA-16792
> Project: Kafka
> Issue Type: Test
> Components: clients, consumer, unit tests
> Reporter: Lianet Magrans
> Assignee: PoAn Yang
> Priority: Blocker
> Labels: kip-848-client-support
> Fix For: 4.0.0
>
>
> Enable the following unit tests for the new async consumer in
> KafkaConsumerTest:
> - testCurrentLag
> - testFetchStableOffsetThrowInPoll
> - testListOffsetShouldUpdateSubscriptions
> - testPollReturnsRecords
> - testResetToCommittedOffset
> - testResetUsingAutoResetPolicy
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
