| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Partially addresses #47424.
|
|
|
|
|
|
|
|
|
| |
This reverts the addition of the "goldiloader" Gem and all use of it.
While this Gem is very promising it's causing a variety of problems on
GitLab.com due to it eager-loading too much data in places where we
don't expect/can handle this. At least for the time being this means we
have to go back to manually fixing N+1 query problems, but at least
those should not cause a negative impact on availability.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Goldiloader is great, but has several issues with has_many :through relations:
* https://github.com/salsify/goldiloader/issues/12
* https://github.com/salsify/goldiloader/issues/14
* https://github.com/salsify/goldiloader/issues/18
Rather than try to figure out which applies in each case, we should just do the
drudge work of manually disabling autoloading for all relations of this type. We
can always use regular preloading for specific cases, but this way we avoid
generating invalid queries through Goldiloader's magic.
|
|
|
|
|
| |
We need to close all merge requests coming from the project within the
entire fork network.
|
| |
|
|
|
|
|
| |
When no fork network exists for the source projects, we create a new
one with the correct source
|
|
The fork network will keep track of the root project as long as it's
present.
|