Clever / thrift-pool

A module that wraps thrift interfaces in connection pooling logic to make them more resilient.
Apache License 2.0
22 stars 15 forks source link

Error in expiration validating of connections #16

Closed towertop closed 6 years ago

towertop commented 8 years ago

When use pool optino ttl, this line mistakes to determine the connection as expired. This make the pool repeatedly destroy and create connections in every cycle.

My service was also suffering a lot of zombie established thrift connections. I'll follow to check if the two problem coupled.

rodrigopr commented 7 years ago

Any update on this? it fixed the TTL bug for us as well.

tupy commented 7 years ago

+1

paulopaquielli commented 7 years ago

+1

marciovicente commented 7 years ago

Nice! I'm waiting for it

helielson commented 7 years ago

+1