Andy McCurdy

Results 28 comments of Andy McCurdy

Setting `socket_timeout` configures the socket to raise a timeout after `socket_timeout` seconds have elapsed. Setting this value too low for your environment will cause every command to a Redis or...

Ya, this is definitely a problem. In my opinion, it's more "correct" to raise an error in this case. We're asking for a decoded response in a specific encoding. Getting...

That's pretty much what you'll be able to do with the context manager I'm working on. It's fairly complicated because of how pipelines work and the hiredis-py integration. Something like:...

@dpopowich Your suggestion might work out some of the time on Python 2 but it will be break many things on Python 3. Many stdlib functions specifically require bytes arguments....

The proposed change (redis/hiredis-py#82) to hiredis-py has been accepted and merged. Future versions of hiredis-py will behave sanely and no longer silently discard decoding errors by default.

@piercefreeman That's still what I would like to do. I just haven't had time to implement it.

@harishkumarr95 I think you're in the wrong thread. Can you open up a new issue and provide a code sample that fails?

There's not enough information for me to reproduce the error that you're seeing. If you can put together a sample app that consistently reproduces the problem then we can likely...

@Hamza-Lachi @silviuum If either of you can provide an example that reproduces this issue I can take a look and try to figure out what's going on. Currently I don't...

@mikeauthramp Is you app running in a multithreaded environment? This seems like a race condition to me. Specifically, I'm wondering if this error is another manifestation of celery/celery#6335.