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

Re-enable kafka-virtual-threads alert-message test #35398

Conversation

ozangunalp
Copy link
Contributor

@ozangunalp ozangunalp commented Aug 17, 2023

Post-ack may run on the caller thread (which is the virtual-thread that executed the method)

It was disabled in 94772f8

@quarkus-bot
Copy link

quarkus-bot bot commented Aug 17, 2023

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should not end up with dot

This message is automatically generated by a bot.

@ozangunalp ozangunalp changed the title Re-enable kafka-virtual-threads alert-message test. Re-enable kafka-virtual-threads alert-message test Aug 17, 2023
@quarkus-bot

This comment has been minimized.

@ozangunalp ozangunalp force-pushed the reenable_kafka_virtual_threads_alertMessage branch from d98bd50 to 27d4d82 Compare August 17, 2023 12:58
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 17, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@ozangunalp ozangunalp requested a review from gsmet August 17, 2023 14:34
@gsmet
Copy link
Member

gsmet commented Aug 17, 2023

@cescoffier I let you check the behavior is correct. Other than that it's fine by me.

@cescoffier cescoffier merged commit 31d78a6 into quarkusio:main Aug 17, 2023
@quarkus-bot quarkus-bot bot added this to the 3.4 - main milestone Aug 17, 2023
@ozangunalp ozangunalp deleted the reenable_kafka_virtual_threads_alertMessage branch August 24, 2023 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants