From 1d4ab0bc51405c7fdf91c72afbbdff5dd18e3889 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Toma=C5=BE=20Jerman?= Date: Mon, 24 Apr 2023 12:49:19 +0200 Subject: [PATCH] "Fix" broken upgrade for dedup config on mssql The upgrade was simply omitted since there is no way it could happen on MSSQL (it was introduced when the change was introduced). --- server/store/adapters/rdbms/upgrade_fixes.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/server/store/adapters/rdbms/upgrade_fixes.go b/server/store/adapters/rdbms/upgrade_fixes.go index 0cc96e856..ce7a79c60 100644 --- a/server/store/adapters/rdbms/upgrade_fixes.go +++ b/server/store/adapters/rdbms/upgrade_fixes.go @@ -661,6 +661,13 @@ func fix_2022_09_07_changePostgresIdColumnsDatatype(ctx context.Context, s *Stor } func fix_2023_03_00_migrateComposeModuleConfigForRecordDeDup(ctx context.Context, s *Store) (err error) { + // @note skipping this for SQL server since it was introduced with 2023.3.0 so there. + // There are issues with the implementation which won't work on mssql. + // Since there is no way this would do anything on mssql, we can skip it. + if s.DB.DriverName() == "sqlserver" { + return + } + type ( oldRule struct { Name string `json:"name"`