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

Fix a concurrency issue in WeakLazyStripes #3219

Merged

Conversation

@adamretter
Copy link
Member

adamretter commented Jan 22, 2020

A small issue where a delete from stripes was interleaved with a read without locking.

Non-deterministic, and so incredibly hard to test for. Only appears under high concurrency.

@adamretter adamretter requested a review from eXist-db/core Jan 22, 2020
@adamretter adamretter added the bug label Jan 22, 2020
@adamretter adamretter added this to the eXist-5.1.2 milestone Jan 22, 2020
@adamretter adamretter changed the title [bugfix] Fix a concurrency issue in WeakLazyStripes Fix a concurrency issue in WeakLazyStripes Jan 22, 2020
@dizzzz
dizzzz approved these changes Jan 22, 2020
@dizzzz dizzzz merged commit 32fc878 into eXist-db:develop Jan 22, 2020
3 checks passed
3 checks passed
Codacy/PR Quality Review Up to standards. A positive pull request.
Details
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.