Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
c9cd43d9
Commit
c9cd43d9
authored
Feb 15, 2021
by
Jeremy Jackson
Committed by
Alper Akgun
Feb 15, 2021
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove caching_experiments feature flag [RUN ALL RSPEC] [RUN AS-IF-FOSS]
parent
33f7a2bc
Changes
4
Show whitespace changes
Inline
Side-by-side
Showing
4 changed files
with
5 additions
and
21 deletions
+5
-21
app/experiments/application_experiment.rb
app/experiments/application_experiment.rb
+0
-1
changelogs/unreleased/jj-remove-caching-experiments.yml
changelogs/unreleased/jj-remove-caching-experiments.yml
+5
-0
config/feature_flags/development/caching_experiments.yml
config/feature_flags/development/caching_experiments.yml
+0
-8
spec/experiments/application_experiment/cache_spec.rb
spec/experiments/application_experiment/cache_spec.rb
+0
-12
No files found.
app/experiments/application_experiment.rb
View file @
c9cd43d9
...
@@ -108,7 +108,6 @@ class ApplicationExperiment < Gitlab::Experiment # rubocop:disable Gitlab/Namesp
...
@@ -108,7 +108,6 @@ class ApplicationExperiment < Gitlab::Experiment # rubocop:disable Gitlab/Namesp
end
end
def
write_entry
(
key
,
entry
,
**
options
)
def
write_entry
(
key
,
entry
,
**
options
)
return
false
unless
Feature
.
enabled?
(
:caching_experiments
)
return
false
if
entry
.
value
.
blank?
# don't cache any empty values
return
false
if
entry
.
value
.
blank?
# don't cache any empty values
pool
{
|
redis
|
redis
.
hset
(
*
hkey
(
key
),
entry
.
value
)
}
pool
{
|
redis
|
redis
.
hset
(
*
hkey
(
key
),
entry
.
value
)
}
...
...
changelogs/unreleased/jj-remove-caching-experiments.yml
0 → 100644
View file @
c9cd43d9
---
title
:
Remove caching_experiments feature flag
merge_request
:
53901
author
:
type
:
other
config/feature_flags/development/caching_experiments.yml
deleted
100644 → 0
View file @
33f7a2bc
---
name
:
caching_experiments
introduced_by_url
:
https://gitlab.com/gitlab-org/gitlab/-/merge_requests/49669
rollout_issue_url
:
milestone
:
'
13.7'
type
:
development
group
:
group::adoption
default_enabled
:
false
spec/experiments/application_experiment/cache_spec.rb
View file @
c9cd43d9
...
@@ -51,16 +51,4 @@ RSpec.describe ApplicationExperiment::Cache do
...
@@ -51,16 +51,4 @@ RSpec.describe ApplicationExperiment::Cache do
'invalid call to clear a non-hash cache key'
'invalid call to clear a non-hash cache key'
)
)
end
end
context
"when the :caching_experiments feature is disabled"
do
before
do
stub_feature_flags
(
caching_experiments:
false
)
end
it
"doesn't write to the cache"
do
subject
.
write
(
key_field
,
'value'
)
expect
(
subject
.
read
(
key_field
)).
to
be_nil
end
end
end
end
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment