summaryrefslogtreecommitdiff
path: root/app/services/branches
diff options
context:
space:
mode:
authorJohn Cai <jcai@gitlab.com>2019-07-02 10:48:13 -0700
committerJohn Cai <jcai@gitlab.com>2019-07-02 17:07:33 -0700
commit2ceffce190165321a1818801e245456870c4708a (patch)
treec9ed1afe9b18c24f56bc92d2efcfe7abe3c745d2 /app/services/branches
parentc6861b189b7faaf385bd397fd41ba93ed5649f25 (diff)
downloadgitlab-ce-2ceffce190165321a1818801e245456870c4708a.tar.gz
Deprecate diverging commit count with max parameterjc-deprecate-diverging-count-with-max
In 12.0, we turned the feature flag on that effectively turned off the --max-count flag for the count diverging commits call. Since we have commit graphs turned on, this did not affect preformance negatively. Thus, we want to deprecate the call that passes --max-count
Diffstat (limited to 'app/services/branches')
-rw-r--r--app/services/branches/diverging_commit_counts_service.rb24
1 files changed, 1 insertions, 23 deletions
diff --git a/app/services/branches/diverging_commit_counts_service.rb b/app/services/branches/diverging_commit_counts_service.rb
index f947cec1663..a3404caf2d7 100644
--- a/app/services/branches/diverging_commit_counts_service.rb
+++ b/app/services/branches/diverging_commit_counts_service.rb
@@ -8,11 +8,7 @@ module Branches
end
def call(branch)
- if Feature.enabled?('gitaly_count_diverging_commits_no_max')
- diverging_commit_counts_without_max(branch)
- else
- diverging_commit_counts(branch)
- end
+ diverging_commit_counts(branch)
end
private
@@ -22,27 +18,9 @@ module Branches
delegate :raw_repository, to: :repository
def diverging_commit_counts(branch)
- ## TODO: deprecate the below code after 12.0
@root_ref_hash ||= raw_repository.commit(repository.root_ref).id
cache.fetch(:"diverging_commit_counts_#{branch.name}") do
number_commits_behind, number_commits_ahead =
- repository.raw_repository.diverging_commit_count(
- @root_ref_hash,
- branch.dereferenced_target.sha,
- max_count: Repository::MAX_DIVERGING_COUNT)
-
- if number_commits_behind + number_commits_ahead >= Repository::MAX_DIVERGING_COUNT
- { distance: Repository::MAX_DIVERGING_COUNT }
- else
- { behind: number_commits_behind, ahead: number_commits_ahead }
- end
- end
- end
-
- def diverging_commit_counts_without_max(branch)
- @root_ref_hash ||= raw_repository.commit(repository.root_ref).id
- cache.fetch(:"diverging_commit_counts_without_max_#{branch.name}") do
- number_commits_behind, number_commits_ahead =
raw_repository.diverging_commit_count(
@root_ref_hash,
branch.dereferenced_target.sha)