Rescue thrift errors in UdpSender and add logger

salemove / jaeger-client-ruby

OpenTracing Tracer implementation for Jaeger in Ruby

MIT License

62 stars 38 forks source link

Rescue thrift errors in UdpSender and add logger #11

Closed kruczjak closed 6 years ago

kruczjak commented 6 years ago

Hi! I've added this pull request as it may be worth to merging.

Why: We have multiple problems with UdpSender thread dying all the time (and even kill an app which has Thread.abort_on_exception enabled). This additions allows to log errors and skip failed batches.