Don't load all pull requests attributes

Author: lunny

models/migrate.go

@@ -208,7 +208,7 @@ func InsertReleases(rels ...*Release) error {
 	return sess.Commit()
 }
 
-// UpdateReviewsMigrationsByType updates reivews' migrations information via given git service type and original id and poster id
+// UpdateReviewsMigrationsByType updates reviews' migrations information via given git service type and original id and poster id
 func UpdateReviewsMigrationsByType(tp structs.GitServiceType, originalAuthorID string, posterID int64) error {
 	_, err := x.Table("review").
 		Where(builder.In("issue_id",

models/pull.go

@@ -655,6 +655,19 @@ func GetPullRequestByID(id int64) (*PullRequest, error) {
 	return getPullRequestByID(x, id)
 }
 
+// GetPullRequestByIssueIDWithNoAttributes returns pull request with no attributes loaded by given issue ID.
+func GetPullRequestByIssueIDWithNoAttributes(issueID int64) (*PullRequest, error) {
+	var pr PullRequest
+	has, err := x.Where("issue_id = ?", issueID).Get(&pr)
+	if err != nil {
+		return nil, err
+	}
+	if !has {
+		return nil, ErrPullRequestNotExist{0, issueID, 0, 0, "", ""}
+	}
+	return &pr, nil
+}
+
 func getPullRequestByIssueID(e Engine, issueID int64) (*PullRequest, error) {
 	pr := &PullRequest{
 		IssueID: issueID,

modules/migrations/gitea.go

@@ -775,7 +775,7 @@ func (g *GiteaLocalUploader) CreateReviews(reviews ...*base.Review) error {
 		pr, ok := g.prCache[issueID]
 		if !ok {
 			var err error
-			pr, err = models.GetPullRequestByIssueID(issueID)
+			pr, err = models.GetPullRequestByIssueIDWithNoAttributes(issueID)
 			if err != nil {
 				return err
 			}