Commit 65d333c0 authored by James Fargher's avatar James Fargher

Merge branch 'remove_use_distinct_in_shas_cte_ff' into 'master'

Remove the use_distinct_in_shas_cte FF

See merge request gitlab-org/gitlab!65027
parents 9d38f1ce 4068bbd9
...@@ -47,8 +47,7 @@ module Ci ...@@ -47,8 +47,7 @@ module Ci
# rubocop: disable CodeReuse/ActiveRecord # rubocop: disable CodeReuse/ActiveRecord
def pipelines_using_cte def pipelines_using_cte
sha_relation = merge_request.all_commits.select(:sha) sha_relation = merge_request.all_commits.select(:sha).distinct
sha_relation = sha_relation.distinct if Feature.enabled?(:use_distinct_in_shas_cte, default_enabled: :yaml)
cte = Gitlab::SQL::CTE.new(:shas, sha_relation) cte = Gitlab::SQL::CTE.new(:shas, sha_relation)
......
---
name: use_distinct_in_shas_cte
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/61454
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/330586
milestone: '13.12'
type: development
group: group::optimize
default_enabled: true
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment