Skip to content

MINOR: Increase throughput too slow for consumer to read within timeout#6061

Merged
guozhangwang merged 2 commits intoapache:2.1from
bbejeck:MINOR_standby_task_test_throughput_too_low_2_1
Dec 22, 2018
Merged

MINOR: Increase throughput too slow for consumer to read within timeout#6061
guozhangwang merged 2 commits intoapache:2.1from
bbejeck:MINOR_standby_task_test_throughput_too_low_2_1

Conversation

@bbejeck
Copy link
Copy Markdown
Member

@bbejeck bbejeck commented Dec 21, 2018

Previous PR #6043 reduced throughput for VerifiableProducer in base class, but the streams_standby_replica_test needs higher throughput for consumer to complete verification in 60 seconds. Same update as #6060

For testing kicked off branch builder with 25 repeats https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2202/

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Dec 21, 2018

ping @guozhangwang, @mjsax, @vvcephei for reviews

@bbejeck bbejeck force-pushed the MINOR_standby_task_test_throughput_too_low_2_1 branch from 109e05f to d41246e Compare December 22, 2018 00:19
@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Dec 22, 2018

@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Dec 22, 2018

@guozhangwang guozhangwang merged commit b1ea6e0 into apache:2.1 Dec 22, 2018
guozhangwang pushed a commit that referenced this pull request Dec 22, 2018
Previous PR #6043 reduced throughput for VerifiableProducer in base class, but the streams_standby_replica_test needs higher throughput for consumer to complete verification in 60 seconds. Same update as #6060 and #6061

Reviewers: Guozhang Wang <wangguoz@gmail.com>
@bbejeck bbejeck deleted the MINOR_standby_task_test_throughput_too_low_2_1 branch July 10, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants