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

Reactive routes: virtual threads support #36466

Merged
merged 2 commits into from
Oct 14, 2023
Merged

Conversation

mkouba
Copy link
Contributor

@mkouba mkouba commented Oct 13, 2023

Virtual threads: integration tests refactoring

  • move the logic from AssertHelpers to a shared component - VirtualThreadsAssertions located in quarkus-test-vertx

@quarkus-bot

This comment has been minimized.

@mkouba
Copy link
Contributor Author

mkouba commented Oct 13, 2023

So this is funny. You need at least JDK 20 to run the tests but then impsort is disabled. Never forget to switch to JDK 17 to sort the imports 🤦.

@quarkus-bot

This comment has been minimized.

- move the logic from AssertHelpers to a shared component - VirtualThreadsAssertions located in quarkus-test-vertx
@mkouba mkouba added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Oct 13, 2023
@github-actions
Copy link

github-actions bot commented Oct 13, 2023

🙈 The PR is closed and the preview is expired.

@quarkus-bot

This comment has been minimized.

@quarkus-bot
Copy link

quarkus-bot bot commented Oct 14, 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.

@mkouba mkouba merged commit ec4074a into quarkusio:main Oct 14, 2023
26 checks passed
@quarkus-bot quarkus-bot bot added the kind/enhancement New feature or request label Oct 14, 2023
@quarkus-bot quarkus-bot bot added this to the 3.6 - main milestone Oct 14, 2023
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Virtual-Threads Support for quarkus-reactive-routes
2 participants