diff options
| author | Jan Provaznik <jprovaznik@gitlab.com> | 2019-09-06 11:54:58 +0200 |
|---|---|---|
| committer | Jan Provaznik <jprovaznik@gitlab.com> | 2019-09-07 06:43:17 +0200 |
| commit | f2754e416e16c1b56810b835505346c295578c7a (patch) | |
| tree | 42867740b585563f6569e5b9a46071c67466e5bc /app/models/merge_request.rb | |
| parent | 093858adc76f893c35b0bcf25127c3f229a7bcb6 (diff) | |
| download | gitlab-ce-f2754e416e16c1b56810b835505346c295578c7a.tar.gz | |
Use delete_all for deleting eventsevents-delete-all
Because we don't have any destroy callbacks (or other logic
triggered on event destroy), there is no reason for deleting events
inefficiently one by one, instead we can use :delete_all.
Diffstat (limited to 'app/models/merge_request.rb')
| -rw-r--r-- | app/models/merge_request.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb index 95daa48d4bc..901ebcf249f 100644 --- a/app/models/merge_request.rb +++ b/app/models/merge_request.rb @@ -54,7 +54,7 @@ class MergeRequest < ApplicationRecord belongs_to :head_pipeline, foreign_key: "head_pipeline_id", class_name: "Ci::Pipeline" - has_many :events, as: :target, dependent: :destroy # rubocop:disable Cop/ActiveRecordDependent + has_many :events, as: :target, dependent: :delete_all # rubocop:disable Cop/ActiveRecordDependent has_many :merge_requests_closing_issues, class_name: 'MergeRequestsClosingIssues', |