dlecocq / nsq-py

Hacking on Python NSQ Bindings
MIT License
44 stars 16 forks source link

Reduce logging around http clients #26

Closed dlecocq closed 10 years ago

dlecocq commented 10 years ago

I accidentally let some logging trickle into the HTTP clients, so now I'm paring it back a bit. Logging was a performance issue at one point in hot sections (in particular nsq.connection.Connection.read), but the HTTP clients aren't performance-critical and the logging isn't in a hot loop, so a small amount of logging is reasonable.

b4hand commented 10 years ago

LGTM