Skip to content

Commit f69c849

Browse files
authored
Reverse order of values when upserting (#1317)
1 parent 2e23939 commit f69c849

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
- [#1301](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1301) Add support for INDEX INCLUDE.
66
- [#1312](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1312) Add support for `insert_all` and `upsert_all`
7+
- [#1317](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1317) Reverse order of values when upserting
78

89
#### Changed
910

lib/active_record/connection_adapters/sqlserver/database_statements.rb

+2
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,8 @@ def build_insert_sql(insert) # :nodoc:
169169
end
170170

171171
def build_sql_for_merge_insert(insert:, insert_all:, columns_with_uniqueness_constraints:) # :nodoc:
172+
insert_all.inserts.reverse! if insert.update_duplicates?
173+
172174
sql = <<~SQL
173175
MERGE INTO #{insert.model.quoted_table_name} WITH (UPDLOCK, HOLDLOCK) AS target
174176
USING (

0 commit comments

Comments
 (0)