Commit 3d94dd84 authored by nmilojevic1's avatar nmilojevic1

Add comment about allwoing n+1 calls

parent 7b1f1869
...@@ -129,6 +129,11 @@ class GitlabProjectImport ...@@ -129,6 +129,11 @@ class GitlabProjectImport
def with_isolated_sidekiq_job def with_isolated_sidekiq_job
Sidekiq::Testing.fake! do Sidekiq::Testing.fake! do
with_request_store do with_request_store do
# If you are attempting to import a large project into a development environment,
# you may see Gitaly throw an error about too many calls or invocations.
# This is due to a n+1 calls limit being set for development setups (not enforced in production)
# https://gitlab.com/gitlab-org/gitlab/-/merge_requests/24475#note_283090635
# For development setups, this code-path will be excluded from n+1 detection.
::Gitlab::GitalyClient.allow_n_plus_1_calls do ::Gitlab::GitalyClient.allow_n_plus_1_calls do
measurement_enabled? ? with_measuring { yield } : yield measurement_enabled? ? with_measuring { yield } : yield
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment