Commit c05af01c authored by Alex Pooley's avatar Alex Pooley

Removed sql_set_operators feature flag

Union set operator now matches intersect and except set operators.
parent e69eba78
...@@ -37,27 +37,6 @@ module FromUnion ...@@ -37,27 +37,6 @@ module FromUnion
# rubocop: disable Gitlab/Union # rubocop: disable Gitlab/Union
extend FromSetOperator extend FromSetOperator
define_set_operator Gitlab::SQL::Union define_set_operator Gitlab::SQL::Union
alias_method :from_union_set_operator, :from_union
def from_union(members, remove_duplicates: true, alias_as: table_name)
if Feature.enabled?(:sql_set_operators)
from_union_set_operator(members, remove_duplicates: remove_duplicates, alias_as: alias_as)
else
# The original from_union method.
standard_from_union(members, remove_duplicates: remove_duplicates, alias_as: alias_as)
end
end
private
def standard_from_union(members, remove_duplicates: true, alias_as: table_name)
union = Gitlab::SQL::Union
.new(members, remove_duplicates: remove_duplicates)
.to_sql
from(Arel.sql("(#{union}) #{alias_as}"))
end
# rubocop: enable Gitlab/Union # rubocop: enable Gitlab/Union
end end
end end
---
title: Enable refactored union set operator
merge_request: 46295
author:
type: added
---
name: sql_set_operators
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/39786
rollout_issue_url:
group: group::access
type: development
default_enabled: false
...@@ -3,13 +3,5 @@ ...@@ -3,13 +3,5 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe FromUnion do RSpec.describe FromUnion do
[true, false].each do |sql_set_operator| it_behaves_like 'from set operator', Gitlab::SQL::Union
context "when sql-set-operators feature flag is #{sql_set_operator}" do
before do
stub_feature_flags(sql_set_operators: sql_set_operator)
end
it_behaves_like 'from set operator', Gitlab::SQL::Union
end
end
end end
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