aboutsummaryrefslogtreecommitdiffstats
path: root/models/migrations/v136.go
diff options
context:
space:
mode:
author6543 <6543@obermui.de>2020-04-14 15:53:34 +0200
committerGitHub <noreply@github.com>2020-04-14 09:53:34 -0400
commit10e2f291442fdc7efc31c02f5ffcba79a36db9ac (patch)
tree5dbda60a983caf8d83ad7202755587acd692c212 /models/migrations/v136.go
parentc571c5bb286b925297f4eb32d130a5496126c3cb (diff)
downloadgitea-10e2f291442fdc7efc31c02f5ffcba79a36db9ac.tar.gz
gitea-10e2f291442fdc7efc31c02f5ffcba79a36db9ac.zip
Cache PullRequest Divergence (#10914)
* Cache PullRequest Divergence * only re-calc divergence if AddTestPullRequestTask() is exec * migrate already open pulls * finalize * take care of closed&not-merged+deleted-branch pull requests * fix nil pointer exeption Signed-off-by: 6543 <6543@obermui.de> * try this * no error its a warn * init gitea-repositories-meta * dont use gitDivergence type * CI.restart() * CI.restart() * CI.restart() * CI.restart() * check IsUserAllowedToUpdate independend from CommitsBehind
Diffstat (limited to 'models/migrations/v136.go')
-rw-r--r--models/migrations/v136.go69
1 files changed, 69 insertions, 0 deletions
diff --git a/models/migrations/v136.go b/models/migrations/v136.go
new file mode 100644
index 0000000000..115861f344
--- /dev/null
+++ b/models/migrations/v136.go
@@ -0,0 +1,69 @@
+// Copyright 2020 The Gitea Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package migrations
+
+import (
+ "fmt"
+
+ "code.gitea.io/gitea/models"
+ "code.gitea.io/gitea/modules/log"
+ "code.gitea.io/gitea/modules/setting"
+ pull_service "code.gitea.io/gitea/services/pull"
+
+ "xorm.io/xorm"
+)
+
+func addCommitDivergenceToPulls(x *xorm.Engine) error {
+
+ if err := x.Sync2(new(models.PullRequest)); err != nil {
+ return fmt.Errorf("Sync2: %v", err)
+ }
+
+ var last int
+ batchSize := setting.Database.IterateBufferSize
+ sess := x.NewSession()
+ defer sess.Close()
+ for {
+ if err := sess.Begin(); err != nil {
+ return err
+ }
+ var results = make([]*models.PullRequest, 0, batchSize)
+ err := sess.Where("has_merged = ?", false).OrderBy("id").Limit(batchSize, last).Find(&results)
+ if err != nil {
+ return err
+ }
+ if len(results) == 0 {
+ break
+ }
+ last += len(results)
+
+ for _, pr := range results {
+ divergence, err := pull_service.GetDiverging(pr)
+ if err != nil {
+ if err = pr.LoadIssue(); err != nil {
+ return fmt.Errorf("pr.LoadIssue()[%d]: %v", pr.ID, err)
+ }
+ if !pr.Issue.IsClosed {
+ return fmt.Errorf("GetDiverging: %v", err)
+ }
+ log.Warn("Could not recalculate Divergence for pull: %d", pr.ID)
+ pr.CommitsAhead = 0
+ pr.CommitsBehind = 0
+ }
+ if divergence != nil {
+ pr.CommitsAhead = divergence.Ahead
+ pr.CommitsBehind = divergence.Behind
+ }
+ if _, err = sess.ID(pr.ID).Cols("commits_ahead", "commits_behind").Update(pr); err != nil {
+ return fmt.Errorf("Update Cols: %v", err)
+ }
+ }
+
+ if err := sess.Commit(); err != nil {
+ return err
+ }
+ }
+ return nil
+}