summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Rename use_conditions_only option to use_where_in.42877-snippets-dashboard-slowAndreas Brandl2018-02-202-4/+4
|
* Avoid manual SQL for visibility level check.Andreas Brandl2018-02-201-1/+1
|
* Cleanup access level shortcut.Andreas Brandl2018-02-203-50/+18
|
* Simplify down migration.Andreas Brandl2018-02-201-1/+1
|
* Add changelog.Andreas Brandl2018-02-201-0/+5
| | | | Closes #42877.
* Allow choice between #where or #from.Andreas Brandl2018-02-202-4/+12
| | | | | | | | | | | | Immediately using #from here requires a lot of changes in other finders (e.g. IssuableFinder, TodosFinder). In all places where we use #merge, this goes completely the wrong way when passed in a relation that was built with `#from(...)`: The original query's FROM part gets completely replaced. This avoids changing all other places and focuses on improving SnippetFinder with the downside of two (small) codepaths to do the same thing.
* Don't mess with table aliases.Andreas Brandl2018-02-201-4/+4
|
* Remove duplication in Project methods.Andreas Brandl2018-02-202-38/+21
|
* Push feature-related query part up.Andreas Brandl2018-02-202-2/+30
|
* Add partial index on projects for index-only scans.Andreas Brandl2018-02-202-1/+23
| | | | | | | | | | | | | | This helps with queries that get project ids based on the - comparably rare - visibility levels 10 and 20. For these, postgres can now leverage the partial index for a index-only scan to improve performance. Example queries: SELECT id FROM projects WHERE visibility_level IN (10,20) SELECT id FROM projects WHERE visibility_level IN (10) For MySQL, this results in a full index on id because MySQL omits the WHERE clause. That is, the index is a duplicate of the primary key basically.
* Shortcut when all levels visible.Andreas Brandl2018-02-203-1/+34
|
* Replace OR clause with UNION.Andreas Brandl2018-02-201-1/+7
|
* Merge branch 'move-spintach-project-network-graph-to-rspec' into 'master'Rémy Coutable2018-02-203-162/+108
|\ | | | | | | | | Move spinach project network graph tests to RSpec See merge request gitlab-org/gitlab-ce!17171
| * Move spinach project network graph tests to RSpecSemyon Pupkov2018-02-163-162/+108
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/issues/23036
* | Merge branch 'remove-unnecessary-validate-project' into 'master'Kamil Trzciński2018-02-203-3/+8
|\ \ | | | | | | | | | | | | Remove unecessary validate: true from belongs_to :project See merge request gitlab-org/gitlab-ce!17166
| * | Remove unecessary validate: true from belongs_to :projectremove-unnecessary-validate-projectDylan Griffith2018-02-203-3/+8
| | | | | | | | | | | | This does not seem to serve any clear purpose and causes other issues (see https://gitlab.com/gitlab-org/gitlab-ce/issues/43196#note_59275652)
* | | Merge branch 'remove-chart-show-bundle' into 'master'Phil Hughes2018-02-2011-29/+9
|\ \ \ | | | | | | | | | | | | | | | | Remove graphs_show.js webpack bundle See merge request gitlab-org/gitlab-ce!17194
| * | | fix broken specsremove-chart-show-bundleMike Greiling2018-02-194-5/+5
| | | |
| * | | remove common_d3 bundleMike Greiling2018-02-193-15/+0
| | | |
| * | | remove graphs_show webpack bundleMike Greiling2018-02-196-9/+4
| | | |
* | | | Merge branch 'rs-over-testing' into 'master'Rémy Coutable2018-02-192-50/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove extraneous tests from Issues API spec Closes #25201 See merge request gitlab-org/gitlab-ce!17222
| * | | | Remove extraneous tests from Issues API specrs-over-testingRobert Speicher2018-02-192-50/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These were all testing an implementation detail of `Issues::UpdateService` which is already well-tested and doesn't need to be re-tested here. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/25201
* | | | | Merge branch 'docs/rs-changelog-doc-links' into 'master'Rémy Coutable2018-02-191-2/+2
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | Update two links in the changelog developer docs See merge request gitlab-org/gitlab-ce!17220
| * | | | Update two links in the changelog developer docsdocs/rs-changelog-doc-linksRobert Speicher2018-02-181-2/+2
| | | | |
* | | | | Merge branch 'chart.html.haml-refactor' into 'master'43411-update-postgresql-ha-docsClement Ho2018-02-1937-3533/+51
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | Chart.html.haml refactor See merge request gitlab-org/gitlab-ce!16897
| * | | | Chart.html.haml refactorJacob Schatz2018-02-1937-3533/+51
|/ / / /
* | | | Merge branch '4826-github-import-wiki-fix-1' into 'master'Nick Thomas2018-02-193-2/+12
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [GH Import] Create an empty wiki if wiki import failed Closes #43020 See merge request gitlab-org/gitlab-ce!17112
| * | | | [GH Import] Create an empty wiki if wiki import failedValery Sizov2018-02-193-2/+12
|/ / / /
* | | | Merge branch '42274-group-request-membership-long-too' into 'master'Rémy Coutable2018-02-195-78/+114
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "group request membership mail with too long list of "To:"" Closes #42274 See merge request gitlab-org/gitlab-ce!17121
| * | | | Resolve "group request membership mail with too long list of "To:""🙈 jacopo beschi 🙉2018-02-195-78/+114
|/ / / /
* | | | Merge branch 'kp-fix-stacked-bar-progress-value-clipping' into 'master'Filipa Lacerda2018-02-192-1/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix single digit value clipping for stacked progress bar Closes gitlab-ee#4864 See merge request gitlab-org/gitlab-ce!17217
| * | | | Add changelog entrykp-fix-stacked-bar-progress-value-clippingKushal Pandya2018-02-191-0/+5
| | | | |
| * | | | Fix single digit value clippingKushal Pandya2018-02-191-1/+1
|/ / / /
* | | | Merge branch 'jej/avoid-slow-file-lock-checks' into 'master'Sean McGivern2018-02-193-5/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid slow File Lock checks when not used Closes #43339 See merge request gitlab-org/gitlab-ce!17140
| * | | | Avoid slow File Lock checks when not usedJames Edwards-Jones2018-02-193-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | | Also avoid double commit lookup during file lock check by reusing memoized commits.
* | | | | Merge branch 'flipper-caching' into 'master'Douwe Maan2018-02-192-1/+6
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Increase feature flag cache TTL to one hour Closes #40854 See merge request gitlab-org/gitlab-ce!17213
| * | | | | Increase feature flag cache TTL to one hourYorick Peterse2018-02-192-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Flipper already takes care of flushing cache entries when enabling/disabling features so it should be safe to increase the TTL. This in turn should drastically reduce the number of Flipper queries executed. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/40854
* | | | | | Merge branch 'qa/gb/fix-tests-for-hidden-secret-variables' into 'master'Rémy Coutable2018-02-195-40/+34
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix and simplify end-to-end tests for secret variables Closes gitlab-qa#190 See merge request gitlab-org/gitlab-ce!17173
| * | | | | Remove useless assignment in secret variables specsqa/gb/fix-tests-for-hidden-secret-variablesGrzegorz Bizon2018-02-161-1/+1
| | | | | |
| * | | | | Fix and simplify end-to-end tests for secret variablesGrzegorz Bizon2018-02-165-39/+33
| | |_|/ / | |/| | |
* | | | | Merge branch '32564-fix-double-system-closing-notes' into 'master'Sean McGivern2018-02-195-25/+51
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Double closing system notes when closing issue with Merge Request" Closes #32546 and #32564 See merge request gitlab-org/gitlab-ce!17035
| * | | | | Convert Gitaly commit parent IDs to array as early as possibleSean McGivern2018-02-191-10/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tracking issue if this causes problems is https://gitlab.com/gitlab-org/gitaly/issues/1028
| * | | | | applying feedback32564-fix-double-system-closing-notesMicaël Bergeron2018-02-163-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | # modified: lib/gitlab/git/commit.rb
| * | | | | fix specsMicaël Bergeron2018-02-162-13/+18
| | | | | |
| * | | | | add changelogMicaël Bergeron2018-02-161-0/+5
| | | | | |
| * | | | | stop ProcessCommitWorker from processing MR merge commitMicaël Bergeron2018-02-163-17/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a merge request is merged, it creates a commit with the description of the MR, which may contain references and issue closing references. As this will be handled in the PostMergeService anyways, let's ignore merge commit generated from a MR.
* | | | | | Merge branch 'sh-guard-read-only-user-updates' into 'master'Nick Thomas2018-02-193-0/+15
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't attempt to update user tracked fields if database is in read-only Closes gitlab-ee#4972 See merge request gitlab-org/gitlab-ce!17200
| * | | | | | Don't attempt to update user tracked fields if database is in read-onlysh-guard-read-only-user-updatesStan Hu2018-02-173-0/+15
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With Geo, attempting to view an endpoint with a user could result in an Error 500 since Devise attempts to update the last sign-in IP and other details. Closes gitlab-org/gitlab-ee#4972
* | | | | | Merge branch 'docs-articles-layout-frontmatter' into 'master'Achilleas Pipinellis2018-02-1911-63/+90
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docs: Update all articles with the new layout (metadata from the frontmatter) Closes #42831 See merge request gitlab-org/gitlab-ce!17187
| * | | | | | Update all articles with the new layout (meta data from the frontmatter)docs-articles-layout-frontmatterMarcia Ramos2018-02-1611-63/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Context: https://gitlab.com/gitlab-com/gitlab-docs/merge_requests/182