Redis pubsub result #1
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
regen does not work with dequeue(regen(it)), as deque seems to use some C magic instead of __iter__ for copying the list, so the iterator ends up being consumed.
…ng threads. Closes #2567
…t process. Closes #2510
Update CONTRIBUTORS.txt
Fixed bug where database backend not passing through URL to BaseBackend __init__.
Database backend url passing
Getting rid of leaking memory + adding minlen size of the set minlen is minimal residual size of set after operating for long. Minlen items are kept, even if they should be expired by time, until we get newer items. Problems with older and even more old code: 1) Heap would tend to grow in some scenarios (like adding an item multiple times). 2) Adding many items fast would not clean them soon enough (if ever). 3) When talking to other workers, revoked._data was sent, but it was processed on the other side as iterable. That means giving those keys new (current) timestamp. By doing this workers could recycle items forever. Combined with 1) and 2), this means that in large set of workers, you are getting out of memory soon. All those problems should be fixed now, also some new unittests are added. This should fix issues #3095, #3086.
Database backend url passing
…nc backend interface (Issue #2511)
…edis-pubsub-result
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To combine.