inst_statsd 3.2.0 → 3.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 705acd8e1a6b843e6f4e54ec0e1525f1165022547c22649d4b0c60c2c1fc83fa
-  data.tar.gz: 1d0daad7330f761a6ea4642bd52c85a0a4bf5a6856e7bd967750670a7f08832e
+  metadata.gz: 4e7d5fbbb438dc387efee59fe23c96d65d8c6d15b68bf08e3bacfaf1f05c9ac0
+  data.tar.gz: 27f3a13b0e1fe8910a9330b88adac40545e397500dc6d901f8bbcad0014f4cdc
 SHA512:
-  metadata.gz: 1a23712f36838b2fd9afd2fe15430c9239e14b212aa6a05ed3358c81ff9799a2d11345dc1170915de903d6a73eed0a0dce05c183c75661693296ab5511b2cdb8
-  data.tar.gz: d5a2406ba15f33196527a9d24cb7d52d6eb5d0aea387f7a80d9382e389777e0c375473d50282af0841f9832adc8f27d8c89b3f07c249764de3170aa8b033303e
+  metadata.gz: '09adfd59ec9e74f78a0af260d339f7db562a1e281e8b514650d5259a2f6ac85529cecb73b8b5a8e42f0d9f193c3aec8a1049de813280b5b8b2ceca12df261fad'
+  data.tar.gz: 697e4ef4dcd88dc71c7ff550433992dedf39b942bc21298cc45d623c55f223507370ceb6d8ed465cdc01e7a2c0d1719c5672dc338e1a73b777f8f8cf3c291bf2

data/lib/inst_statsd/distribution.rb

@@ -18,6 +18,12 @@ module InstStatsd
     def distribution(metric, value, tags: {})
       return unless instance && data_dog?
 
+      metric = if append_hostname?
+                 "#{metric}.#{hostname}"
+               else
+                 metric.to_s
+               end
+
       instance.distribution(metric, value, { tags: tags.merge(dog_tags) }.compact)
     end
 
@@ -25,11 +31,15 @@ module InstStatsd
     #
     # @param metric [String] The name of the metric to increment.
     # @param tags [Hash] Optional tags to associate with the metric.
+    # @param short_stat [String, nil] Stat name to use instead of `metric` if backed by DataDog.
     #
     # @example Increment the error count:
     #  InstStatsd::Statsd.distributed_increment('client.request.failed', tags: { status: '500' })
-    def distributed_increment(metric, tags: {})
-      distribution(metric, 1, tags: tags)
+    def distributed_increment(metric, tags: {}, short_stat: nil)
+      # Non-Datadog clients don't support distribution metrics, so we use fall back to increment
+      return increment(metric, tags: tags, short_stat: short_stat) if instance && !data_dog?
+
+      distribution(short_stat || metric, 1, tags: tags)
     end
   end
 end

data/lib/inst_statsd/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module InstStatsd
-  VERSION = "3.2.0"
+  VERSION = "3.3.1"
 end

data/spec/inst_statsd/distribution_spec.rb

@@ -4,19 +4,19 @@ require "spec_helper"
 require "datadog/statsd"
 
 RSpec.describe InstStatsd::Distribution do
+  let(:instance) { instance_double(Datadog::Statsd) }
+
+  before do
+    allow(InstStatsd::Statsd).to receive_messages(data_dog?: is_datadog, dog_tags: dog_tags, instance: instance)
+  end
+
   describe ".distribution" do
     subject { InstStatsd::Statsd.distribution(metric, value, tags: tags) }
 
-    let(:instance) { instance_double(Datadog::Statsd) }
-
     let(:metric) { "client.request.failed" }
     let(:value) { 23 }
     let(:tags) { { status: "500" } }
 
-    before do
-      allow(InstStatsd::Statsd).to receive_messages(data_dog?: is_datadog, dog_tags: dog_tags, instance: instance)
-    end
-
     context "when instance and data_dog? are configured" do
       let(:dog_tags) { { environment: "production" } }
       let(:is_datadog) { true }
@@ -49,6 +49,8 @@ RSpec.describe InstStatsd::Distribution do
 
     let(:metric) { "client.request.failed" }
     let(:tags) { { status: "500" } }
+    let(:dog_tags) { { status: "401" } }
+    let(:is_datadog) { true }
 
     before do
       allow(InstStatsd::Statsd).to receive_messages(distribution: nil)
@@ -59,5 +61,16 @@ RSpec.describe InstStatsd::Distribution do
 
       subject
     end
+
+    context "when the configured instance is not a DataDog instance" do
+      let(:dog_tags) { {} }
+      let(:is_datadog) { false }
+
+      it 'invokes "increment" on InstStatsd::Statsd with metric and tags' do
+        expect(InstStatsd::Statsd).to receive(:increment).with(metric, tags: tags, short_stat: nil)
+
+        subject
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inst_statsd
 version: !ruby/object:Gem::Version
-  version: 3.2.0
+  version: 3.3.1
 platform: ruby
 authors:
 - Nick Cloward
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-12-09 00:00:00.000000000 Z
+date: 2025-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aroi