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

Timeout when the reader cannot consume messages anymore #196

Closed
mostafa opened this issue Feb 6, 2023 · 3 comments · Fixed by #246
Closed

Timeout when the reader cannot consume messages anymore #196

mostafa opened this issue Feb 6, 2023 · 3 comments · Fixed by #246
Assignees
Labels
🐛 Bug Something isn't working 🧑‍💻 Good First Issue Good for newcomers

Comments

@mostafa
Copy link
Owner

mostafa commented Feb 6, 2023

Due to several reports in #180, I need to introduce an external context with timeout to prevent reader.consume from hanging.

@JasmineCA
Copy link
Contributor

JasmineCA commented May 24, 2023

Hello @mostafa. First, thanks for your extension! Currently this is the only one to work with kafka in k6 script! Do you plan to work on this bug soon? This is very annoying in my project when some kafka messages are not received randomly, the test hangs until k6 reaches the end of the test duration time. I'm not really skilled with writting extensions for kafka yet (only wrote an extension to do some custom crypto operations and keep keys) so definitely I have no idea about adding a timeout in the context to make it work.

@JasmineCA
Copy link
Contributor

Hello @mostafa , do you have any update on this?

@mostafa
Copy link
Owner Author

mostafa commented Aug 4, 2023

Hey @JasmineCA,

I haven't had time to implement this, yet I highly appreciate if anybody does.

@mostafa mostafa added the 🧑‍💻 Good First Issue Good for newcomers label Aug 4, 2023
@mostafa mostafa linked a pull request Sep 14, 2023 that will close this issue
@github-project-automation github-project-automation bot moved this from Todo to Test in xk6-kafka Sep 14, 2023
@mostafa mostafa moved this from Test to Release in xk6-kafka Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working 🧑‍💻 Good First Issue Good for newcomers
Projects
Status: Release
Development

Successfully merging a pull request may close this issue.

2 participants