diff options
author | Ahmad Sherif <me@ahmadsherif.com> | 2018-02-07 12:04:49 +0100 |
---|---|---|
committer | Ahmad Sherif <me@ahmadsherif.com> | 2018-02-07 13:46:55 +0100 |
commit | e2e11a05afd260f136ced4811d467f8f5935e928 (patch) | |
tree | 149e78dc962f838167cb44e3e7903980c104053f | |
parent | 826105dfda91081e92dfd5ca7f85a1343893e049 (diff) | |
download | gitlab-ce-fix/merge-requests-create-service-n-plus-1.tar.gz |
-rw-r--r-- | app/services/merge_requests/create_service.rb | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/app/services/merge_requests/create_service.rb b/app/services/merge_requests/create_service.rb index 634bf3bd690..46dc59bcec3 100644 --- a/app/services/merge_requests/create_service.rb +++ b/app/services/merge_requests/create_service.rb @@ -1,6 +1,7 @@ module MergeRequests class CreateService < MergeRequests::BaseService def execute + RequestStore.begin! set_projects! merge_request = MergeRequest.new @@ -9,10 +10,10 @@ module MergeRequests merge_request.source_branch = params[:source_branch] merge_request.merge_params['force_remove_source_branch'] = params.delete(:force_remove_source_branch) - # n+1: https://gitlab.com/gitlab-org/gitlab-ce/issues/37439 - Gitlab::GitalyClient.allow_n_plus_1_calls do - create(merge_request) - end + a = create(merge_request) + RequestStore.end! + RequestStore.clear! + a end def before_create(merge_request) |