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
PAYARA-2677 Reset thread-context ClassLoader after de-serialization for Payara 4 #2623
PAYARA-2677 Reset thread-context ClassLoader after de-serialization for Payara 4 #2623
Conversation
jenkins quick test payara 4 |
Quick build and test passed! |
f2e74ce
to
4a40d34
Compare
jenkins quick test payara 4 |
Quick build and test passed! |
@@ -68,6 +68,19 @@ public void close() { | |||
private final InvocationManager invMgr; | |||
} | |||
|
|||
@RequiredArgsConstructor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove lombok, update copyright
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copyright updated.
Once again, you can't just "remove lombok" I will create a massive PR that replaces lombok with boilerplate and then an informed decision can be made.
For now, Lombok stays. Can't hold up a regression issue for this
jenkins quick test payara 4 |
Quick build and test passed! |
fixes #2604 for Payara 4