diff options
| author | Douwe Maan <douwe@gitlab.com> | 2016-12-02 07:32:23 +0300 |
|---|---|---|
| committer | Douwe Maan <douwe@gitlab.com> | 2016-12-02 07:32:23 +0300 |
| commit | ec4e7d9a9f7f3526a15905c3fb5d851e4c173349 (patch) | |
| tree | a67d8d342dc0e317a38014fb1c93de771bb137db /spec/requests | |
| parent | 71ba28e43967b5eaf0499c2a93c0b1d4f78a35f5 (diff) | |
| parent | 6b4d33566f5f434cc86381a4a1347e42bbe348ee (diff) | |
Merge branch 'process-commit-worker-improvements' into 'master'
Pass commit data to ProcessCommitWorker
This changes `ProcessCommitWorker` so that it takes a Hash containing commit data instead of a commit SHA. This means the worker doesn't have to access Git just to process a commit message (and other data it may use). This in turn should solve the problem of ending up with 15 000-something jobs in the `process_commit` queue that take forever to process.
See merge request !7744
Diffstat (limited to 'spec/requests')
| -rw-r--r-- | spec/requests/projects/cycle_analytics_events_spec.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/requests/projects/cycle_analytics_events_spec.rb b/spec/requests/projects/cycle_analytics_events_spec.rb index 5c90fd9bad9..f5e0fdcda2d 100644 --- a/spec/requests/projects/cycle_analytics_events_spec.rb +++ b/spec/requests/projects/cycle_analytics_events_spec.rb @@ -135,6 +135,6 @@ describe 'cycle analytics events' do merge_merge_requests_closing_issue(issue) - ProcessCommitWorker.new.perform(project.id, user.id, mr.commits.last.sha) + ProcessCommitWorker.new.perform(project.id, user.id, mr.commits.last.to_hash) end end |