don't manage redis connections in test #228

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@dpickett

as alluded to in #123, allow users of zeus to self
manage their redis connections in test.

don't manage redis connections in test
as alluded to in #123, allow users of zeus to self
manage their redis connections in test.
@turadg

This comment has been minimized.

Show comment Hide comment
@turadg

turadg Feb 2, 2013

Collaborator

I can merge this but I don't use Redis and don't know whether this is an improvement. Can other Redis users weigh in?

Is this still necessary since 4009bf3 ?

Collaborator

turadg commented Feb 2, 2013

I can merge this but I don't use Redis and don't know whether this is an improvement. Can other Redis users weigh in?

Is this still necessary since 4009bf3 ?

@dpickett

This comment has been minimized.

Show comment Hide comment
@dpickett

dpickett Feb 2, 2013

I'm normally not a fan of catch all exceptions, but 4009bf3 solves the problem I was having. I'm going to close this ticket, thanks!

dpickett commented Feb 2, 2013

I'm normally not a fan of catch all exceptions, but 4009bf3 solves the problem I was having. I'm going to close this ticket, thanks!

@turadg turadg closed this Feb 16, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment