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

Remove unnecessary synchronization around Tablet.closeState #1037

Closed
keith-turner opened this issue Mar 15, 2019 · 0 comments

Comments

@keith-turner
Copy link
Contributor

commented Mar 15, 2019

The Tablet class has a closeState variable and a family of associated methods that read it. These methods are synchronized and this seems to be unnecessary because the variable is volatile. The synchronization has a sharp edge as seen in #1031. Need to analyze the usage of all the methods and ensure it safe to remove synchronization and if so remove it.

keith-turner added a commit to keith-turner/accumulo that referenced this issue Apr 11, 2019

@ctubbsii ctubbsii added the v2.0.0 label Jun 14, 2019

@ctubbsii ctubbsii added this to To do in 2.0.0 via automation Jun 14, 2019

@ctubbsii ctubbsii moved this from To do to Done in 2.0.0 Jun 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
2 participants
You can’t perform that action at this time.