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

Replace volatile usage with VarHandle release/acquire for better performance #4598

Open
hmottestad opened this issue May 24, 2023 · 0 comments · May be fixed by #4599
Open

Replace volatile usage with VarHandle release/acquire for better performance #4598

hmottestad opened this issue May 24, 2023 · 0 comments · May be fixed by #4599
Assignees
@hmottestad
Labels
⏩ performance
Milestone
4.3.12

Comments

@hmottestad
Copy link
Contributor

We typically use volatile for variables that can be updated and read by multiple threads. Unless we need total order for read/write to multiple variables then it is may be safe to use release/acquire (RA.

See:
@hmottestad hmottestad added this to the 4.3.2 milestone May 24, 2023
@hmottestad hmottestad self-assigned this May 24, 2023
@hmottestad hmottestad linked a pull request May 24, 2023 that will close this issue
Draft
5 tasks
@hmottestad hmottestad modified the milestones: 4.3.2, 4.3.3 Jun 6, 2023
@hmottestad hmottestad modified the milestones: 4.3.3, 4.3.4 Jul 5, 2023
@hmottestad hmottestad modified the milestones: 4.3.4, 4.3.5 Jul 22, 2023
@hmottestad hmottestad modified the milestones: 4.3.5, 4.3.6 Aug 6, 2023
@hmottestad hmottestad modified the milestones: 4.3.6, 4.3.7 Aug 27, 2023
@hmottestad hmottestad modified the milestones: 4.3.7, 4.3.8 Oct 6, 2023
@hmottestad hmottestad modified the milestones: 4.3.8, 4.3.9 Nov 7, 2023
@hmottestad hmottestad modified the milestones: 4.3.9, 4.3.10 Jan 21, 2024
@hmottestad hmottestad modified the milestones: 4.3.10, 4.3.11 Mar 6, 2024
@hmottestad hmottestad modified the milestones: 4.3.11, 4.3.12 Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hmottestad hmottestad

Labels
⏩ performance
Projects
None yet
Milestone
4.3.12
Development

Successfully merging a pull request may close this issue.

Replace volatile with release/acquire where possible eclipse-rdf4j/rdf4j
1 participant
@hmottestad