Skip to content

MINOR: standby task test throughput too low 2.0#6062

Merged
guozhangwang merged 4 commits intoapache:2.0from
bbejeck:MINOR_standby_task_test_throughput_too_low_2_0
Dec 22, 2018
Merged

MINOR: standby task test throughput too low 2.0#6062
guozhangwang merged 4 commits intoapache:2.0from
bbejeck:MINOR_standby_task_test_throughput_too_low_2_0

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 and #6061

Kicked off branch builder with 25 repeats https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2203/

Committer Checklist (excluded from commit message)

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

@bbejeck bbejeck changed the title Minor standby task test throughput too low 2 0 MINOR: standby task test throughput too low 2 0 Dec 21, 2018
@bbejeck bbejeck changed the title MINOR: standby task test throughput too low 2 0 MINOR: standby task test throughput too low 2.0 Dec 21, 2018
@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Dec 21, 2018

ping @guozhangwang, @mjsax, and @vvcephei for reviews

@bbejeck bbejeck force-pushed the MINOR_standby_task_test_throughput_too_low_2_0 branch from e5e7022 to ae30baf Compare December 22, 2018 00:17
@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 38fd13d into apache:2.0 Dec 22, 2018
@bbejeck bbejeck deleted the MINOR_standby_task_test_throughput_too_low_2_0 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