diff options
author | Andreas Brandl <abrandl@gitlab.com> | 2019-09-05 13:00:30 +0300 |
---|---|---|
committer | Andreas Brandl <abrandl@gitlab.com> | 2019-09-05 13:00:30 +0300 |
commit | c099fa810ff24bb3f9860569d6e1cfce92508468 (patch) | |
tree | 07d87c87d62d6b7802dde8ccf7d0831918df8f8d /app | |
parent | 4e9c531a84c012d2947e39d483531895d02208ea (diff) | |
parent | 56c01f62d5c5509f63aa46547cf90276a360775e (diff) |
Merge branch 'bjk/routable_counter' into 'master'
Adjust routable metric
See merge request gitlab-org/gitlab-ce!32591
Diffstat (limited to 'app')
-rw-r--r-- | app/models/concerns/routable.rb | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/app/models/concerns/routable.rb b/app/models/concerns/routable.rb index 07d22641a0a..8b011bca72c 100644 --- a/app/models/concerns/routable.rb +++ b/app/models/concerns/routable.rb @@ -33,7 +33,7 @@ module Routable # # Returns a single object, or nil. def find_by_full_path(path, follow_redirects: false) - increment_counter(:routable_find_by_full_path, 'Number of calls to Routable.find_by_full_path') + routable_calls_counter.increment(method: 'find_by_full_path') if Feature.enabled?(:routable_two_step_lookup) # Case sensitive match first (it's cheaper and the usual case) @@ -61,7 +61,7 @@ module Routable def where_full_path_in(paths) return none if paths.empty? - increment_counter(:routable_where_full_path_in, 'Number of calls to Routable.where_full_path_in') + routable_calls_counter.increment(method: 'where_full_path_in') wheres = paths.map do |path| "(LOWER(routes.path) = LOWER(#{connection.quote(path)}))" @@ -71,12 +71,8 @@ module Routable end # Temporary instrumentation of method calls - def increment_counter(counter, description) - @counters[counter] ||= Gitlab::Metrics.counter(counter, description) - - @counters[counter].increment - rescue - # ignore the error + def routable_calls_counter + @routable_calls_counter ||= Gitlab::Metrics.counter(:gitlab_routable_calls_total, 'Number of calls to Routable by method') end end |