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

Remove legacy TCCL check log #4036

Closed
vietj opened this issue Jul 27, 2021 · 0 comments
Closed

Remove legacy TCCL check log #4036

vietj opened this issue Jul 27, 2021 · 0 comments
Assignees
@vietj
Labels
enhancement
Milestone
4.2.0

Comments

@vietj
Copy link
Member

vietj commented Jul 27, 2021 •
edited
Loading

This log is not really helpful, in addition it is possible to disable TCCL and have a TCCL set to the non system classloader.
@vietj vietj added this to the 4.2.0 milestone Jul 27, 2021
@vietj vietj self-assigned this Jul 27, 2021
vietj added a commit that referenced this issue Jul 27, 2021
@vietj
Add a VertxOptions 'disableTCCL' that configures whether Vertx will s…
e2b0b40 
…et the thread context classloader when running actions on a Vert.x context. This options is set by default to the legacy 'vertx.disableTCCL' system property.

The warn logged when the TCCL is disabled and the current thread classloader is not the system classloader is removed.

fixes #4035
fixes #4036
@vietj vietj closed this as completed in 417919b Jul 27, 2021
vietj added a commit that referenced this issue Jul 27, 2021
@vietj
Add a VertxOptions 'disableTCCL' that configures whether Vertx will s…
cf4c6c2 
…et the thread context classloader when running actions on a Vert.x context. This options is set by default to the legacy 'vertx.disableTCCL' system property.

The warn logged when the TCCL is disabled and the current thread classloader is not the system classloader is removed.

fixes #4035
fixes #4036
vietj added a commit that referenced this issue Jul 27, 2021
@vietj
Add a VertxOptions 'disableTCCL' that configures whether Vertx will s…
4bd363b 
…et the thread context classloader when running actions on a Vert.x context. This options is set by default to the legacy 'vertx.disableTCCL' system property.

The warn logged when the TCCL is disabled and the current thread classloader is not the system classloader is removed.

fixes #4035
fixes #4036
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vietj vietj

Labels
enhancement
Projects
None yet
Milestone
4.2.0
Development

No branches or pull requests
1 participant
@vietj