Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KafkaSinkCluster: test multi topic consuming #1786

Merged
merged 2 commits into from
Oct 28, 2024

Conversation

rukai
Copy link
Member

@rukai rukai commented Oct 28, 2024

Adds a test case for consumers that consume from more than one topic at a time.
This was an attempt to reproduce #1764 but while it did not reproduce the issue, it is still worth landing the extra test case for the improved coverage.

Copy link

codspeed-hq bot commented Oct 28, 2024

CodSpeed Performance Report

Merging #1786 will not alter performance

Comparing rukai:test_multi_topic_consuming (14e48b1) with main (830187d)

Summary

✅ 38 untouched benchmarks

@rukai rukai force-pushed the test_multi_topic_consuming branch 2 times, most recently from cb10e33 to dee5702 Compare October 28, 2024 03:37
@rukai rukai force-pushed the test_multi_topic_consuming branch from dee5702 to 529b819 Compare October 28, 2024 03:38
@rukai rukai enabled auto-merge (squash) October 28, 2024 10:41
@rukai rukai merged commit 487ffec into shotover:main Oct 28, 2024
41 checks passed
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.

3 participants