From cabfac35a2d56b1f6e352e86bdc67382eef2de90 Mon Sep 17 00:00:00 2001
From: Jonathan Hyman <jon@braze.com>
Date: Mon, 26 Apr 2021 17:17:13 -0400
Subject: [PATCH] Remove RTT checking from Sidekiq since we have our own
 monitoring, this takes up unnecessary ops/second on Redis.

---
 lib/sidekiq/launcher.rb | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/lib/sidekiq/launcher.rb b/lib/sidekiq/launcher.rb
index ae529fd49..3fc160b47 100644
--- 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)