diff options
author | Stan Hu <stanhu@gmail.com> | 2019-07-02 16:58:47 +0300 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2019-07-02 17:26:27 +0300 |
commit | 385aa46046ec83e6837c106576699f76e65876a7 (patch) | |
tree | b602ce89be24b9c8f2e04f18f6ccde58f2d6ae35 | |
parent | e07ebe66af957c46e7c69329b3ab561bb539351b (diff) |
Cache Flipper persisted names directly to local memory storage
Now that application settings are no longer dominating network traffic,
we see that the Feature#persisted_names is using a significant amount of
CPU and network bandwidth for Redis. Move this cache into the
thread-local memory storage to reduce Redis overhead.
-rw-r--r-- | changelogs/unreleased/sh-cache-flipper-names-memory-cache.yml | 5 | ||||
-rw-r--r-- | lib/feature.rb | 4 | ||||
-rw-r--r-- | spec/lib/feature_spec.rb | 2 |
3 files changed, 9 insertions, 2 deletions
diff --git a/changelogs/unreleased/sh-cache-flipper-names-memory-cache.yml b/changelogs/unreleased/sh-cache-flipper-names-memory-cache.yml new file mode 100644 index 00000000000..00443e81244 --- /dev/null +++ b/changelogs/unreleased/sh-cache-flipper-names-memory-cache.yml @@ -0,0 +1,5 @@ +--- +title: Cache Flipper persisted names directly to local memory storage +merge_request: 30265 +author: +type: performance diff --git a/lib/feature.rb b/lib/feature.rb index cc9c9d44005..22420e95ea2 100644 --- a/lib/feature.rb +++ b/lib/feature.rb @@ -34,7 +34,9 @@ class Feature begin # We saw on GitLab.com, this database request was called 2300 # times/s. Let's cache it for a minute to avoid that load. - Rails.cache.fetch('flipper:persisted_names', expires_in: 1.minute) { FlipperFeature.feature_names } + Gitlab::ThreadMemoryCache.cache_backend.fetch('flipper:persisted_names', expires_in: 1.minute) do + FlipperFeature.feature_names + end end end diff --git a/spec/lib/feature_spec.rb b/spec/lib/feature_spec.rb index 6f05914f915..403e0785d1b 100644 --- a/spec/lib/feature_spec.rb +++ b/spec/lib/feature_spec.rb @@ -40,7 +40,7 @@ describe Feature do .once .and_call_original - expect(Rails.cache) + expect(Gitlab::ThreadMemoryCache.cache_backend) .to receive(:fetch) .once .with('flipper:persisted_names', expires_in: 1.minute) |