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

Race condition in workload D? #327

Closed
uncle-betty opened this issue Jun 26, 2015 · 2 comments
Closed

Race condition in workload D? #327

uncle-betty opened this issue Jun 26, 2015 · 2 comments

Comments

@uncle-betty
Copy link
Contributor

There seems to be a race condition in workload D.

The client threads generate keys for the records to be inserted and there is a small time window between the thread generating the key and it actually inserting the record. However, as soon as the key has been generated, that key is considered to be available for other threads to read. (Because transactioninsertkeysequence.lastInt() now returns that key.) That results in the other threads potentially trying to read records that don't yet exist.
@allanbank allanbank mentioned this issue Jul 7, 2015
Closed
@allanbank allanbank mentioned this issue Jul 19, 2015
Closed
@uncle-betty
Copy link
Contributor Author

Proposed fix in pull request #364.

@uncle-betty uncle-betty mentioned this issue Jul 19, 2015
Merged
allanbank added a commit to allanbank/YCSB that referenced this issue Jul 22, 2015
@allanbank
Remove synchronized from the acknowledge method and avoid queue lock …
067373a 
…contention. brianfrankcooper#327
@allanbank allanbank mentioned this issue Aug 23, 2015
Closed
allanbank added a commit that referenced this issue Sep 21, 2015
@allanbank
Merge pull request #364 from tlopatic/insert-fix
51f5633 
Proposed fix for #327.
@allanbank allanbank mentioned this issue Sep 21, 2015
Closed
@allanbank
Copy link
Collaborator

Not sure why it re-openned. Closing again.

jaricftw pushed a commit to jaricftw/YCSB that referenced this issue Jul 19, 2016
@allanbank
Merge pull request brianfrankcooper#364 from tlopatic/insert-fix
9213e53 
Proposed fix for brianfrankcooper#327.
jaricftw pushed a commit to jaricftw/YCSB that referenced this issue Jul 19, 2016
@allanbank
Merge pull request brianfrankcooper#364 from tlopatic/insert-fix
ba04ab2 
Proposed fix for brianfrankcooper#327.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@allanbank @uncle-betty