Zeger-Jan van de Weg
f9565e3039
Batchload blobs for diff generation
...
After installing a new gem, batch-loader, a construct can be used to
queue data to be fetched in bulk. The gem was also introduced in both
gitlab-org/gitlab-ce!14680 and gitlab-org/gitlab-ce!14846 , but those mrs
are not merged yet.
For the generation of diffs, both the old blob and the new blob need to
be loaded. This for every file in the diff, too. Now we collect all
these so we do 1 fetch. Three `.allow_n_plus_1_calls` have been removed,
which I expect to be valid, but this needs to be confirmed by a full CI
run.
Possibly closes:
- https://gitlab.com/gitlab-org/gitlab-ce/issues/37445
- https://gitlab.com/gitlab-org/gitlab-ce/issues/37599
- https://gitlab.com/gitlab-org/gitlab-ce/issues/37431
2017-11-21 13:53:26 +01:00
Douwe Maan
fec48c6e17
Use Commit#notes and Note.for_commit_id when possible to make sure we use all the indexes available to us
2017-11-08 12:22:11 +01:00
micael.bergeron
cd88fa8f80
removed the #ensure_ref_fetched from all controllers
...
also, I refactored the MergeRequest#fetch_ref method to express
the side-effect that this method has.
MergeRequest#fetch_ref -> MergeRequest#fetch_ref!
Repository#fetch_source_branch -> Repository#fetch_source_branch!
2017-11-03 08:13:11 -04:00
Alejandro Rodríguez
faa9bd402d
Create a Gitlab::Git submodule for conlict-related files
...
Rename classes to (hopefully) clearer names while we're doing that.
2017-10-12 22:03:15 -03:00
Alejandro Rodríguez
3fcab51ebb
Refactor conflict resolution to contain git ops within Gitlab::Git
...
This prepares the codebase for a Gitaly migration. See
https://gitlab.com/gitlab-org/gitaly/issues/553
2017-10-12 22:03:14 -03:00
Douwe Maan
546b18b903
Merge branch 'tc-geo-read-only-idea' into 'master'
...
Create idea of read-only database
Closes #37534
See merge request gitlab-org/gitlab-ce!14688
2017-10-07 11:01:21 +00:00
Bob Van Landuyt
7c00b53812
Find branches in all projects in the fork network
2017-10-07 11:46:23 +02:00
Toon Claes
d13669716a
Create idea of read-only database
...
In GitLab EE, a GitLab instance can be read-only (e.g. when it's a Geo
secondary node). But in GitLab CE it also might be useful to have the
"read-only" idea around. So port it back to GitLab CE.
Also having the principle of read-only in GitLab CE would hopefully
lead to less errors introduced, doing write operations when there
aren't allowed for read-only calls.
Closes gitlab-org/gitlab-ce#37534 .
2017-10-06 22:37:40 +02:00
Luke "Jared" Bennett
ff7b545c8e
Merge remote-tracking branch 'origin/master' into 18608-lock-issues
2017-09-20 15:23:00 +01:00
Andrew Newdigate
64d7ec0a9e
Detect n+1 issues involving Gitaly
2017-09-19 10:55:37 +00:00
Jarka Kadlecova
b928720852
Support discussion locking in the backend
2017-09-14 14:50:32 +02:00
Douwe Maan
523a1c69ab
Merge branch '23079-remove-default-scope-in-sortable' into 'master'
...
Removes default scope from sortable
Closes #23079
See merge request !13558
2017-09-07 16:15:32 +00:00
Tiago Botelho
cfd475a45e
Removes default scope from sortable
2017-09-07 13:01:59 +01:00
Douwe Maan
1c55b57175
Merge branch '34509-improves-markdown-rendering-performance-for-commits-list' into 'master'
...
Resolve "Projects::CommitsController#show is slow partially due to SQL queries"
Closes #34509
See merge request !13762
2017-09-06 17:14:09 +00:00
Tiago Botelho
6d8e102c74
Adds cacheless render to Banzai object render
2017-09-06 15:35:35 +01:00
micael.bergeron
966b1128d8
WIP: refactor the first-contributor to Issuable
...
this will remove the need make N queries (per-note) at the
cost of having to mark notes with an attribute
this opens up the possibility for other special roles for notes
2017-09-06 09:00:57 -04:00
Douwe Maan
87cb74a7bc
Make sure MergeRequest commit methods work when either compare_commits or merge_request_diff is set
2017-07-13 18:53:11 -07:00
Luke "Jared" Bennett
c53f2d8ebc
Revert parent param removal
2017-07-07 14:57:28 +01:00
Luke "Jared" Bennett
b8deafd0d8
Removed merge_request/issue state_event param parent and removed issuable_path helpers in favour of issuable_url helpers
2017-07-07 14:57:00 +01:00
Douwe Maan
fe13f11041
Create and use project path helpers that only need a project, no namespace
2017-07-05 11:11:59 -05:00
Douwe Maan
70b05a8377
Split up MergeRequestsController
2017-06-28 18:17:44 -05:00