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 in-memory queue using shared-cache #32

Closed
wants to merge 2 commits into from

Conversation

peter-wangxu
Copy link
Owner

@peter-wangxu peter-wangxu commented Nov 1, 2017

When share in-memory queue in multi-thread, the queue is not
properly synced between threads, this commit uses shared-cache
to achieve the synchronization.
fixes #31

@codecov
Copy link

codecov bot commented Nov 1, 2017

Codecov Report

Merging #32 into master will decrease coverage by 0.68%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage    93.4%   92.71%   -0.69%     
==========================================
  Files           6        6              
  Lines         394      398       +4     
  Branches       48       48              
==========================================
+ Hits          368      369       +1     
- Misses         11       16       +5     
+ Partials       15       13       -2
Flag Coverage Δ
#python 92.71% <66.66%> (-0.69%) ⬇️
Impacted Files Coverage Δ
persistqueue/sqlbase.py 90.9% <66.66%> (-2.78%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a2c4d3...540e0e6. Read the comment docs.

@peter-wangxu peter-wangxu force-pushed the memory_shared_cache branch 2 times, most recently from 6cd79ad to dc576f4 Compare November 1, 2017 08:26
When share in-memory queue in multi-thread, the queue is not
properly synced between threads, this commit uses shared-cache
to achieve the synchronization.
fixes #31
@peter-wangxu peter-wangxu force-pushed the memory_shared_cache branch 3 times, most recently from 39d1e3f to e74130f Compare November 6, 2017 06:15
@peter-wangxu
Copy link
Owner Author

Reopen it if in-memory sqlite queue is needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

in-memory queue is broken
1 participant