Age | Commit message (Collapse) | Author |
|
Support presigned multipart uploads
See merge request gitlab-org/gitlab-ce!18855
|
|
Remove N+1 query for author in issues API
See merge request gitlab-org/gitlab-ce!19345
|
|
Bring back the EE changes to CE to authentication of builds
See merge request gitlab-org/gitlab-ce!19391
|
|
This currently causes 500's errors when loading the MR page
(Discussion) in a few scenarios.
We were not considering detailed diff headers such as
"--- a/doc/update/mysql_to_postgresql.md\n+++ b/doc/update/mysql_to_postgresql.md"
to crop the diff. In order to address it, we're now using
Gitlab::Diff::Parser, clean the diffs and builds Gitlab::Diff::Line objects
we can iterate and filter on.
|
|
This was being masked by the statement cache because only one author was used
per issue in the test..
Also adds support for an Rspec matcher `exceed_all_query_limit`.
|
|
|
|
|
|
|
|
Rails5 fix arel from
Closes #46281
See merge request gitlab-org/gitlab-ce!19340
|
|
|
|
|
|
Fix N+1 with source_projects in merge requests API
See merge request gitlab-org/gitlab-ce!19346
|
|
Now that we are checking `MergeRequest#for_fork?`, we also need
the source project preloaded for a merge request.
|
|
|
|
|
|
|
|
cleared between requests
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
'master'
Improve validations for Ci::Runner#runner_type
See merge request gitlab-org/gitlab-ce!18901
|
|
|
|
Resolve "A lot of constants redefinition warnings"
Closes #46481
See merge request gitlab-org/gitlab-ce!19286
|
|
because of SSRF
|
|
Which could extend from EE
|
|
* Replace `require` or `require_relative` with `require_dependency`
* Remove unneeded `autoload`
|
|
|
|
|
|
Make ActiveRecordSubscriber rails 5 compatible
Closes #44702
See merge request gitlab-org/gitlab-ce!19276
|
|
|
|
Take two for MR metrics population background migration
See merge request gitlab-org/gitlab-ce!19097
|
|
|
|
|
|
Eliminate cached N+1 queries for projects in Issue API
See merge request gitlab-org/gitlab-ce!19269
|
|
Replace .having with .where in calendar query
Closes #46964
See merge request gitlab-org/gitlab-ce!19230
|
|
Fix encoding bug in Gitaly::Commit::TreeEntry
Closes #46445 and gitaly#1208
See merge request gitlab-org/gitlab-ce!19243
|
|
|
|
|
|
|
|
Removal of API v3 from the codebase
Closes #36819
See merge request gitlab-org/gitlab-ce!18970
|
|
Fix various bugs related to relative_url_root in development
See merge request gitlab-org/gitlab-ce!19213
|
|
In CE, every `Issue` entity is also a `ProjectEntity`, which calls
`entity&.project.try(:id)` to show the project ID. In an API
request with 100 issues, this would hit the Rails statement cache 100 times
for the same project and cause unnecessary overhead as related models would
also be loaded.
In EE, we call `Issue#supports_weight?` for each issue, which then calls
`project&.feature_available?(:issue_weights)`. If the project is not
preloaded, this incurs additional overhead, as each individual Project
object has to be queried. This can lead to a significant performance hit.
In loading the CE project with 100 issues, this contributed to at least 22%
of the load time.
See https://gitlab.com/gitlab-org/gitlab-ce/issues/47031 for why testing this
is a bit tricky.
|
|
Add missing usage_ping_enabled to API settings
See merge request gitlab-org/gitlab-ce!19263
|
|
the current syntax doesn't work properly in Rails 5, the resulting query
looks like:
HAVING "events"."project_id" IN (0)
instead of:
HAVING "events"."project_id" IN (SELECT "projects"."id" FROM...
Also we should not use ActiveRecord internal methods. In this case we
can filter projects in WHERE clause instead of doing this in HAVING
clause. Usage of WHERE should be also more efficient because grouping
is then done on much smaller subset of records.
|
|
Identified while resolving conflicts in https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/5906
|
|
Line profiling modal width
Closes #46999
See merge request gitlab-org/gitlab-ce!19253
|
|
|
|
|
|
|
|
|
|
Only preload member records for the relevant projects/groups/user in projects API
See merge request gitlab-org/gitlab-ce!18945
|
|
Replace grape-route-helpers with our own grape-path-helpers
Closes #45718
See merge request gitlab-org/gitlab-ce!19240
|