Remove RTT checking from Sidekiq since we have our own monitoring, th…

…is takes up unnecessary ops/second on Redis.
braze-inc · Apr 26, 2021 · cabfac3 · cabfac3
1 parent 55b7e9d
commit cabfac3
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/sidekiq/launcher.rb b/lib/sidekiq/launcher.rb
@@ -159,7 +159,10 @@ def ❤
           end
         end
 
-        rtt = check_rtt
+        # BRAZE MODIFICATION
+        # We don't care about Sidekiq checking our RTT, it adds unnecessary PING commands which somehow
+        # are slower than any other command.
+        # rtt = check_rtt
 
         fails = procd = 0
         kb = memory_usage(::Process.pid)
@@ -171,7 +174,7 @@ def ❤
             conn.hmset(key, "info", to_json,
               "busy", curstate.size,
               "beat", Time.now.to_f,
-              "rtt_us", rtt,
+              "rtt_us", 0,
               "quiet", @done,
               "rss", kb)
             conn.expire(key, 60)