Skip to content

Fixes: offline_queue and option max_attempts don't work together as expected #280#671

Closed
qdb wants to merge 1 commit into
redis:masterfrom
qdb:master
Closed

Fixes: offline_queue and option max_attempts don't work together as expected #280#671
qdb wants to merge 1 commit into
redis:masterfrom
qdb:master

Conversation

@qdb

@qdb qdb commented Sep 30, 2014

Copy link
Copy Markdown

Fiexes: #280 and #533

Make sure offline queue does not get flushed on subsequent reconnect attempts.

Fixes: redis#280
ee88164 
Make sure offline queue does not get flushed on subsequent reconnect attempts.
@qdb qdb changed the title Fixes: https://github.com/mranney/node_redis/issues/280 Fixes: offline_queue and option max_attempts don't work together as expected #280 Sep 30, 2014
@BridgeAR BridgeAR mentioned this pull request Sep 10, 2015
Merged
BridgeAR added a commit that referenced this pull request Sep 16, 2015
@BridgeAR
Merge pull request #829 from fintura/broken-mode
57b5c7f 
Implement redis connection broken mode and more shiny things

Fixes #569
Fixes #587
Fixes #566 
Fixes #586 
Fixes #280 

This includes the fixes as suggested in #671, #615 and #533. Thx a lot to @qdb, @tobek and @chrishamant 

Closes #675, #463, #362, #438 and #724
@BridgeAR

BridgeAR commented Sep 16, 2015

Copy link
Copy Markdown
Contributor

Thx a lot for your work @qdb !

@BridgeAR BridgeAR closed this Sep 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@qdb @BridgeAR