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

common-util: replacing synchronized classes with their unsynchronized counter parts #3524

Merged
merged 3 commits into from Jan 30, 2019

Conversation

Projects
None yet
6 participants
@svendiedrichsen
Copy link
Contributor

svendiedrichsen commented Dec 11, 2018

No description provided.

@mulderbaba

This comment has been minimized.

Copy link
Member

mulderbaba commented Dec 28, 2018

jenkins test

@mulderbaba mulderbaba added this to the 5.191 milestone Dec 28, 2018

@arjantijms arjantijms requested a review from MeroRai Jan 23, 2019

@svendiedrichsen

This comment has been minimized.

Copy link
Contributor Author

svendiedrichsen commented Jan 23, 2019

@MeroRai I have fixed the code according to your reviews. Also added latest master changes.

@arjantijms

This comment has been minimized.

Copy link
Member

arjantijms commented Jan 24, 2019

Jenkins test please

@MeroRai

This comment has been minimized.

Copy link
Contributor

MeroRai commented Jan 24, 2019

@svendiedrichsen, thanks for responding to all the reviews. As always thank you for your contributions.

@arjantijms arjantijms requested review from Cousjava and MarkWareham Jan 24, 2019

@svendiedrichsen

This comment has been minimized.

Copy link
Contributor Author

svendiedrichsen commented Jan 24, 2019

@MeroRai As always. My pleasure! ;)

@arjantijms arjantijms requested a review from Pandrex247 Jan 28, 2019

@arjantijms arjantijms merged commit 6be8a21 into payara:master Jan 30, 2019

1 check passed

Payara Quick Build and Test Quick build and test passed!
Details

@svendiedrichsen svendiedrichsen deleted the svendiedrichsen:common-util-synchronized branch Jan 30, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.