Skip to content

MINOR: Speed up ShareConsumerTest.testAcquisitionLockTimeoutOnConsumer#22537

Open
Russole wants to merge 2 commits into
apache:trunkfrom
Russole:KAFKA-20674
Open

MINOR: Speed up ShareConsumerTest.testAcquisitionLockTimeoutOnConsumer#22537
Russole wants to merge 2 commits into
apache:trunkfrom
Russole:KAFKA-20674

Conversation

@Russole

@Russole Russole commented Jun 10, 2026

Copy link
Copy Markdown

Summary

Speed up ShareConsumerTest.testAcquisitionLockTimeoutOnConsumer by
overriding the broker-side share record lock duration for this test.

The test previously relied on the class-level
group.share.record.lock.duration.ms=15000 and waited 20000ms for the
acquisition lock to expire. This PR lowers the test-specific lock
duration to 5000ms and reduces the sleep to 8000ms.

Both group.share.min.record.lock.duration.ms and
group.share.record.lock.duration.ms are set because the actual lock
duration must be greater than or equal to the configured minimum.

Reviewers: Chia-Ping Tsai
chia7712@gmail.com

Testing

Ran the target test repeatedly to check for flakiness:

./gradlew :clients:clients-integration-tests:test --tests
org.apache.kafka.clients.consumer.ShareConsumerTest.testAcquisitionLockTimeoutOnConsumer
--rerun-tasks
Result: 10/10 passed.

@github-actions github-actions Bot added triage PRs from the community tests Test fixes (including flaky tests) clients small Small PRs labels Jun 10, 2026
@Russole

Russole commented Jun 10, 2026

Copy link
Copy Markdown
Author

Hi @chia7712 , could you please take a look when you have time? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved clients small Small PRs tests Test fixes (including flaky tests) triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants