From 4fe2ff6e7f17d438346bb9778229960dbfae377d Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Wed, 8 May 2024 12:18:03 +0800 Subject: [PATCH 1/4] Remove If Exist check on migration for mssql because that syntax required SQL server 2016 --- models/migrations/v1_22/v286.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/models/migrations/v1_22/v286.go b/models/migrations/v1_22/v286.go index f46d494dfe6e4..e11d16f8de335 100644 --- a/models/migrations/v1_22/v286.go +++ b/models/migrations/v1_22/v286.go @@ -36,9 +36,9 @@ func expandHashReferencesToSha256(x *xorm.Engine) error { if setting.Database.Type.IsMSSQL() { // drop indexes that need to be re-created afterwards droppedIndexes := []string{ - "DROP INDEX IF EXISTS [IDX_commit_status_context_hash] ON [commit_status]", - "DROP INDEX IF EXISTS [UQE_review_state_pull_commit_user] ON [review_state]", - "DROP INDEX IF EXISTS [UQE_repo_archiver_s] ON [repo_archiver]", + "DROP INDEX [IDX_commit_status_context_hash] ON [commit_status]", + "DROP INDEX [UQE_review_state_pull_commit_user] ON [review_state]", + "DROP INDEX [UQE_repo_archiver_s] ON [repo_archiver]", } for _, s := range droppedIndexes { _, err := db.Exec(s) From e53467c8c01483868721217fdc8340043885495e Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Fri, 10 May 2024 11:58:19 +0800 Subject: [PATCH 2/4] Fix bug --- models/migrations/v1_22/v286_test.go | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/models/migrations/v1_22/v286_test.go b/models/migrations/v1_22/v286_test.go index 7c353747e3f5c..99dbcf68273cb 100644 --- a/models/migrations/v1_22/v286_test.go +++ b/models/migrations/v1_22/v286_test.go @@ -19,21 +19,21 @@ func PrepareOldRepository(t *testing.T) (*xorm.Engine, func()) { type CommitStatus struct { ID int64 - ContextHash string + ContextHash string `xorm:"VARCHAR(40) index"` } type RepoArchiver struct { ID int64 - RepoID int64 - Type int - CommitID string + RepoID int64 `xorm:"index unique(s)"` + Type int `xorm:"unique(s)"` + CommitID string `xorm:"VARCHAR(40) unique(s)"` } type ReviewState struct { ID int64 - CommitSHA string - UserID int64 - PullID int64 + UserID int64 `xorm:"NOT NULL UNIQUE(pull_commit_user)"` + PullID int64 `xorm:"NOT NULL INDEX UNIQUE(pull_commit_user) DEFAULT 0"` + CommitSHA string `xorm:"NOT NULL VARCHAR(40) UNIQUE(pull_commit_user)"` } type Comment struct { From 05c955366a41bbb30dc607f44bfd1c379760502f Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Fri, 10 May 2024 15:02:05 +0800 Subject: [PATCH 3/4] Fix test --- .../migrations/fixtures/Test_RepositoryFormat/review_state.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/models/migrations/fixtures/Test_RepositoryFormat/review_state.yml b/models/migrations/fixtures/Test_RepositoryFormat/review_state.yml index 1197b086e3e92..dd649809164c0 100644 --- a/models/migrations/fixtures/Test_RepositoryFormat/review_state.yml +++ b/models/migrations/fixtures/Test_RepositoryFormat/review_state.yml @@ -1,3 +1,5 @@ - id: 1 + user_id: 1 + pull_id: 1 commit_sha: 19fe5caf872476db265596eaac1dc35ad1c6422d From 8a8e510f0313ec50bf912dc8df7b39ca916d5e44 Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Fri, 10 May 2024 15:07:39 +0800 Subject: [PATCH 4/4] Fix test --- models/migrations/v1_22/v286_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/models/migrations/v1_22/v286_test.go b/models/migrations/v1_22/v286_test.go index 99dbcf68273cb..a19c9396e2e58 100644 --- a/models/migrations/v1_22/v286_test.go +++ b/models/migrations/v1_22/v286_test.go @@ -19,7 +19,7 @@ func PrepareOldRepository(t *testing.T) (*xorm.Engine, func()) { type CommitStatus struct { ID int64 - ContextHash string `xorm:"VARCHAR(40) index"` + ContextHash string `xorm:"char(40) index"` } type RepoArchiver struct {