network_resiliency 0.8.0 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a2a0b6dd58bb024a582f951fd5aa8105506a8c2ffee8dddac284128a09fc844a
-  data.tar.gz: 7b978707f7788382cca10421190942c7395423d363eb222ddb0e5d1f5f1d734d
+  metadata.gz: 0aa3bd17753ae7b79982ef78ec1789f8f83ccbd540165b5600a8d68a521ced06
+  data.tar.gz: c7305e3f42e0229bf63ebc228def5ad136d07539050ea1cf6de21203e5d1656c
 SHA512:
-  metadata.gz: ae2e38bf103409e575f31245634fca438992e9a50415b5c1d0bfa16e75016513a378ad2f0a9c2adef3c22a6392ef2ed1d9da67c26fff745b799bad2c83806e74
-  data.tar.gz: aad0e97f32f64ecb254ee5095375119bf0ff307ab578a7566280e93994c6a59ea3eb3d135a2e1a89bef4ae3f90d4ab94af149dd274594c9f1bcda76f777c2b0e
+  metadata.gz: b6202ad464c14c7632f2bc8b2220a1964a9065deeb0cf6a948bfdbe8e910449bb9f41a26101fa2b35df0a2282f0377a5886d7ca90c260fe98333ebe81b2217c6
+  data.tar.gz: 7aac06c18cb3a2301a7e47f1deae8da37cdbe9c1e975af15c501c150023319cf16225f62b2aae9646222f32a104720fd45a0544d82f9057bcf2f699a3dab19ff

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+###  v0.8.2  (2024-03-13)
+- revert ddog sampling
+- Revert "lower secondary timeout"
+
+###  v0.8.1  (2024-03-12)
+- lower secondary timeout
+
 ###  v0.8.0  (2024-03-12)
 - secondary timeout
 - cover http socket error

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    network_resiliency (0.8.0)
+    network_resiliency (0.8.2)
 
 GEM
   remote: https://rubygems.org/

data/lib/network_resiliency/stats_engine.rb

@@ -65,13 +65,11 @@ module NetworkResiliency
           empty: data.empty?,
           truncated: data.size < dirty_keys.size,
         }.select { |_, v| v },
-        sample_rate: SAMPLE_RATE[:sync],
       )
 
       NetworkResiliency.statsd&.distribution(
         "network_resiliency.sync.keys.dirty",
         dirty_keys.select { |_, n| n > 0 }.count,
-        sample_rate: SAMPLE_RATE[:sync],
       )
 
       return [] if data.empty?

data/lib/network_resiliency/version.rb

@@ -1,3 +1,3 @@
 module NetworkResiliency
-  VERSION = "0.8.0"
+  VERSION = "0.8.2"
 end

data/lib/network_resiliency.rb

@@ -22,11 +22,6 @@ module NetworkResiliency
   DEFAULT_TIMEOUT_MIN = 10 # ms
   MODE = [ :observe, :resilient ].freeze
   RESILIENCY_THRESHOLD = 100
-  SAMPLE_RATE = {
-    timeout: 0.1,
-    stats: 0.1,
-    sync: 0.1,
-  }
 
   extend self
 
@@ -254,7 +249,6 @@ module NetworkResiliency
         adapter: adapter,
         destination: destination,
       },
-      sample_rate: SAMPLE_RATE[:timeout],
     ) if timeout && timeout > 0
 
     if error
@@ -286,28 +280,23 @@ module NetworkResiliency
       # ensure Syncer is running
       Syncer.start
 
-      if rand < SAMPLE_RATE[:stats]
-        NetworkResiliency.statsd&.distribution(
-          "network_resiliency.#{action}.stats.n",
-          stats.n,
-          tags: tags,
-          sample_rate: SAMPLE_RATE[:stats],
-        )
+      NetworkResiliency.statsd&.distribution(
+        "network_resiliency.#{action}.stats.n",
+        stats.n,
+        tags: tags,
+      )
 
-        NetworkResiliency.statsd&.distribution(
-          "network_resiliency.#{action}.stats.avg",
-          stats.avg,
-          tags: tags,
-          sample_rate: SAMPLE_RATE[:stats],
-        )
+      NetworkResiliency.statsd&.distribution(
+        "network_resiliency.#{action}.stats.avg",
+        stats.avg,
+        tags: tags,
+      )
 
-        NetworkResiliency.statsd&.distribution(
-          "network_resiliency.#{action}.stats.stdev",
-          stats.stdev,
-          tags: tags,
-          sample_rate: SAMPLE_RATE[:stats],
-        )
-      end
+      NetworkResiliency.statsd&.distribution(
+        "network_resiliency.#{action}.stats.stdev",
+        stats.stdev,
+        tags: tags,
+      )
     end
 
     nil
@@ -362,20 +351,17 @@ module NetworkResiliency
         # make a second, more lenient attempt
 
         if p99 * 100 < max
-          # max is excessively high
           timeouts << p99 * 100
         elsif p99 * 10 < max
           # use remaining time for second attempt
           timeouts << max - p99
         else
-          # max is smallish
           timeouts << max
 
           NetworkResiliency.statsd&.increment(
             "network_resiliency.timeout.raised",
             tags: tags,
-            sample_rate: SAMPLE_RATE[:timeout],
-          ) if rand < SAMPLE_RATE[:timeout]
+          )
         end
       else
         # the specified timeout is less than our expected p99...awkward
@@ -384,8 +370,7 @@ module NetworkResiliency
         NetworkResiliency.statsd&.increment(
           "network_resiliency.timeout.too_low",
           tags: tags,
-          sample_rate: SAMPLE_RATE[:timeout],
-        ) if rand < SAMPLE_RATE[:timeout]
+        )
       end
     else
       timeouts << p99
@@ -396,8 +381,7 @@ module NetworkResiliency
       NetworkResiliency.statsd&.increment(
         "network_resiliency.timeout.missing",
         tags: tags,
-        sample_rate: SAMPLE_RATE[:timeout],
-      ) if rand < SAMPLE_RATE[:timeout]
+      )
     end
 
     NetworkResiliency.statsd&.distribution(
@@ -407,8 +391,7 @@ module NetworkResiliency
         adapter: adapter,
         destination: destination,
       },
-      sample_rate: SAMPLE_RATE[:timeout],
-    ) if rand < SAMPLE_RATE[:timeout]
+    )
 
     case units
     when nil, :ms, :milliseconds

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: network_resiliency
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.2
 platform: ruby
 authors:
 - Daniel Pepper
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-03-12 00:00:00.000000000 Z
+date: 2024-03-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: byebug