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

Add locks for each Blob and store Stats in a map #219

Merged
merged 12 commits into from
Aug 24, 2021

Conversation

ChristopherHogan
Copy link
Collaborator

  • Maintain a flush_count on VBuckets so we don't destroy one with outstanding flushes.
  • Add a map to the MDM to store Stats and TicketMutex for each Blob. Moving the Stats to a map improves the lookup efficiency from O(n) to O(1). The Blob mutexes are required for implementing asynchronous flushing.
  • Enable unit tests in stb_ds.h for maps with non-string keys. Previously we were only using maps with string keys. Also added a couple bug fixes from upstream.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.3%) to 81.205% when pulling 7a1ee56 on chogan/blob_maps into 29225f6 on master.

@ChristopherHogan ChristopherHogan merged commit fe904f9 into master Aug 24, 2021
@ChristopherHogan ChristopherHogan deleted the chogan/blob_maps branch August 31, 2021 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants