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.

Sign up for GitHub

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

Jump to bottom

Reset the classloader of the internal Vertx pool as well #18944

Merged
merged 1 commit into from
Jul 23, 2021
Merged

Reset the classloader of the internal Vertx pool as well #18944

merged 1 commit into from
Jul 23, 2021

Conversation

Sanne
Copy link
Member

@Sanne Sanne commented Jul 22, 2021

Fixes #18943

This applies the same fix as yesterday to additionally reset the context classloader on the "internal worker pool" of vertx as well.

@quarkus-bot
Copy link

quarkus-bot bot commented Jul 22, 2021

Failing Jobs - Building 9d410c1

Status Name Step Test failures Logs Raw logs
⌛ Native Tests - Spring Build ⚠️ Check → Logs Raw logs

@Sanne Sanne merged commit f0dd4e4 into quarkusio:main Jul 23, 2021
@quarkus-bot quarkus-bot bot added this to the 2.2 - main milestone Jul 23, 2021
@Sanne Sanne deleted the InternalWorkerPoolReset branch July 23, 2021 11:20
@gsmet gsmet modified the milestones: 2.2 - main, 2.1.1.Final Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stuartwdouglas stuartwdouglas stuartwdouglas approved these changes

Assignees
No one assigned
Labels
area/vertx
Projects
None yet
Milestone
2.1.1.Final
Development

Successfully merging this pull request may close these issues.

dev-mode Classloader leak via Vert.x internal worker pool
3 participants
@Sanne @stuartwdouglas @gsmet