diff options
author | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2019-07-24 17:45:01 +0000 |
---|---|---|
committer | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2019-07-24 17:45:01 +0000 |
commit | 6a5d2df3ee53df8d7df84a46c408e4c1fa341f0b (patch) | |
tree | 17411ea4b3c37fe0702db02677a3067b85005b66 /lib/peek/views/redis_detailed.rb | |
parent | d86ef0b00f9a21d07db513d3e4e9311add25075c (diff) | |
parent | d7eadcc0f30d3bd005f9dfb160dd0a460b3a8f56 (diff) | |
download | gitlab-ce-6a5d2df3ee53df8d7df84a46c408e4c1fa341f0b.tar.gz |
Merge branch 'sh-peek-cleanup' into 'master'
Use a base class for Peek views
See merge request gitlab-org/gitlab-ce!31108
Diffstat (limited to 'lib/peek/views/redis_detailed.rb')
-rw-r--r-- | lib/peek/views/redis_detailed.rb | 23 |
1 files changed, 2 insertions, 21 deletions
diff --git a/lib/peek/views/redis_detailed.rb b/lib/peek/views/redis_detailed.rb index 12760c9b75e..c61a1e91282 100644 --- a/lib/peek/views/redis_detailed.rb +++ b/lib/peek/views/redis_detailed.rb @@ -35,36 +35,19 @@ end module Peek module Views - class RedisDetailed < View + class RedisDetailed < DetailedView REDACTED_MARKER = "<redacted>" def key 'redis' end - def results - { - calls: calls, - duration: formatted_duration, - details: details - } - end - def detail_store ::Gitlab::SafeRequestStore['redis_call_details'] ||= [] end private - def formatted_duration - ms = duration * 1000 - if ms >= 1000 - "%.2fms" % ms - else - "%.0fms" % ms - end - end - def duration detail_store.map { |entry| entry[:duration] }.sum # rubocop:disable CodeReuse/ActiveRecord end @@ -73,10 +56,8 @@ module Peek detail_store.count end - def details + def call_details detail_store - .sort { |a, b| b[:duration] <=> a[:duration] } - .map(&method(:format_call_details)) end def format_call_details(call) |