Skip to content

Catch more Exceptions to avoid scheduling poller thread death#353

Merged
mperham merged 1 commit intosidekiq:masterfrom
ncr:patch-1
Aug 16, 2012
Merged

Catch more Exceptions to avoid scheduling poller thread death#353
mperham merged 1 commit intosidekiq:masterfrom
ncr:patch-1

Conversation

@ncr
Copy link
Contributor

@ncr ncr commented Aug 16, 2012

Catch Redis::TimeoutError, Redis::ConnectionError thrown by Redis client on networking errors.

See: https://github.com/redis/redis-rb/blob/master/lib/redis/client.rb#L200

Catch Redis::TimeoutError, Redis::ConnectionError thrown by Redis client on networking errors.

See: https://github.com/redis/redis-rb/blob/master/lib/redis/client.rb#L200
@travisbot
Copy link

This pull request passes (merged aa56400 into 1dc4ddc).

@ncr
Copy link
Contributor Author

ncr commented Aug 16, 2012

Damn, a default commit message... Github makes it too easy to contribute ;)

mperham added a commit that referenced this pull request Aug 16, 2012
Catch more Exceptions to avoid scheduling poller thread death
@mperham mperham merged commit 2655c17 into sidekiq:master Aug 16, 2012
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.

4 participants