Skip to content

Commit

Permalink
Merge pull request #1 from braze-inc/feature-rem-rtt
Browse files Browse the repository at this point in the history
Remove RTT checking from Sidekiq since we have our own monitoring, th…
  • Loading branch information
jonhyman committed Apr 27, 2021
2 parents 55b7e9d + cabfac3 commit f287416
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions lib/sidekiq/launcher.rb
Expand Up @@ -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)
Expand All @@ -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)
Expand Down

0 comments on commit f287416

Please sign in to comment.