Skip to content

Remove synchronized #73

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
Merged
merged 2 commits into from
Jun 12, 2020
Merged

Remove synchronized #73

merged 2 commits into from
Jun 12, 2020

Conversation

@oschwald
Copy link
Member

@oschwald oschwald commented Jun 12, 2020

@coveralls
Copy link

coveralls commented Jun 12, 2020

Coverage Status

Coverage remained the same at 90.939% when pulling 39d0631 on greg/no-synchronized into 95f4c1f on master.

@horgh horgh merged commit 3de5f66 into master Jun 12, 2020
@horgh horgh deleted the greg/no-synchronized branch June 12, 2020 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@horgh horgh horgh approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Huge contention on BufferHolder.get()

4 participants

@oschwald @coveralls @horgh