Repository navigation
Errors in send_stats #259
Description
Hey all,
Firstly, thanks for making such a wonderful piece of software. Appreciate it :)
I'm wondering about the error behavior of send_stats in lib/datadog/statsd.rb, which is called by count, distribution, gauge, histogram, set, and timing.
Specifically, I was wondering whether calling this function can result in an error being raised.
I looked at the implementation of send_stats and noticed that it calls some functions that could raise. For example, send_stats calls forwarder.send_message(full_stat), which calls sender.add(message). The implementation of sender.add seems like it could raise the following error.
def add(message)
raise ArgumentError, 'Start sender first' unless message_queue
... continued ...
end
This issue from 2016 seems to indicate that send_stats is safe (i.e. does not raise an error or call any functions that raise errors), but I wanted to re-confirm the answer.
Thanks!
Activity
remeh commentedon May 31, 2022
Hi @aniruddhb,
You are right that following the code path it looks like it has changed recently (since 5.0.0 I think). How much of this is an issue to you? Do you think that adding this information to methods documentation would be enough?
If this is an issue to you right now, you can use the single thread sender implementation (using
single_thread: trueon statsd constructor) which should never raise an error while usingcount,gaugeand others.aniruddhb commentedon Jun 1, 2022
Hi @remeh
Firstly, thanks for the response. I appreciate your help :)
I don't think its too much of an issue. I am working around it by rescuing errors on the caller side. I was mainly curious whether I could remove this layer of rescuing around the caller or not.
When I get some free time this week, I'll raise a PR to add this information to the method documentation.
Thanks!