diff options
| author | Yorick Peterse <yorickpeterse@gmail.com> | 2015-12-31 17:14:02 +0100 |
|---|---|---|
| committer | Yorick Peterse <yorickpeterse@gmail.com> | 2015-12-31 17:14:02 +0100 |
| commit | a6c60127e3e2966b2f29fa6e6e79503b130c2b02 (patch) | |
| tree | 9f884fb27fd1f6e6c50071e7950b65c8492cfafd /config | |
| parent | ab08321be8405eab07929bb3df8bd2dcc14dc063 (diff) | |
| download | gitlab-ce-a6c60127e3e2966b2f29fa6e6e79503b130c2b02.tar.gz | |
Removed tracking of raw SQL queries
This particular setup had 3 problems:
1. Storing SQL queries as tags is very inefficient as InfluxDB ends up
indexing every query (and they can get pretty large). Storing these
as values instead means we can't always display the SQL as easily.
2. We already instrument ActiveRecord query methods, thus we already
have timing information about database queries.
3. SQL obfuscation is difficult to get right and I'd rather not expose
sensitive data by accident.
Diffstat (limited to 'config')
| -rw-r--r-- | config/initializers/metrics.rb | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/config/initializers/metrics.rb b/config/initializers/metrics.rb index 2e4908192a1..94c535dc562 100644 --- a/config/initializers/metrics.rb +++ b/config/initializers/metrics.rb @@ -7,7 +7,6 @@ if Gitlab::Metrics.enabled? # These are manually require'd so the classes are registered properly with # ActiveSupport. require 'gitlab/metrics/subscribers/action_view' - require 'gitlab/metrics/subscribers/active_record' Gitlab::Application.configure do |config| config.middleware.use(Gitlab::Metrics::RackMiddleware) |