GitLab FOSS merge requestshttps://staging.gitlab.com/gitlab-org/gitlab-foss/-/merge_requests2017-09-19T12:34:56Zhttps://staging.gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/14359WIP: Bridged gitaly exceptions2017-09-19T12:34:56ZAndrew Newdigateandrew@gitlab.comWIP: Bridged gitaly exceptions## What does this MR do?
Reference: https://gitlab.com/gitlab-org/gitaly/issues/588
See https://gitlab.com/gitlab-org/gitaly/merge_requests/358
## Are there points in the code the reviewer needs to double check?
## Why was th...## What does this MR do?
Reference: https://gitlab.com/gitlab-org/gitaly/issues/588
See https://gitlab.com/gitlab-org/gitaly/merge_requests/358
## Are there points in the code the reviewer needs to double check?
## Why was this MR needed?
## Screenshots (if relevant)
## Does this MR meet the acceptance criteria?
- [ ] [Changelog entry](https://docs.gitlab.com/ee/development/changelog.html) added, if necessary
- [ ] [Documentation created/updated](https://docs.gitlab.com/ee/development/doc_styleguide.html)
- [ ] API support added
- [ ] Tests added for this feature/bug
- Review
- [ ] Has been reviewed by UX
- [ ] Has been reviewed by Frontend
- [ ] Has been reviewed by Backend
- [ ] Has been reviewed by Database
- [ ] Conform by the [merge request performance guides](https://docs.gitlab.com/ee/development/merge_request_performance_guidelines.html)
- [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
## What are the relevant issue numbers?Andrew Newdigateandrew@gitlab.comAndrew Newdigateandrew@gitlab.comhttps://staging.gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/14537Removed n+1 exception block to investigate current slowdown2017-10-03T08:10:54ZAndrew Newdigateandrew@gitlab.comRemoved n+1 exception block to investigate current slowdown## What does this MR do?
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/37464
## Are there points in the code the reviewer needs to double check?
## Why was this MR needed?
## Screenshots (if relevant)
## Does this...## What does this MR do?
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/37464
## Are there points in the code the reviewer needs to double check?
## Why was this MR needed?
## Screenshots (if relevant)
## Does this MR meet the acceptance criteria?
- [ ] [Changelog entry](https://docs.gitlab.com/ee/development/changelog.html) added, if necessary
- [ ] [Documentation created/updated](https://docs.gitlab.com/ee/development/doc_styleguide.html)
- [ ] API support added
- [ ] Tests added for this feature/bug
- Review
- [ ] Has been reviewed by UX
- [ ] Has been reviewed by Frontend
- [ ] Has been reviewed by Backend
- [ ] Has been reviewed by Database
- [ ] Conform by the [merge request performance guides](https://docs.gitlab.com/ee/development/merge_request_performance_guidelines.html)
- [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
## What are the relevant issue numbers?https://staging.gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/14652Migrate Workhorse Send{Diff,Patch} to Gitaly2017-10-25T00:00:27Zusername-removed-506061Migrate Workhorse Send{Diff,Patch} to Gitaly~Conversation: https://gitlab.com/gitlab-org/gitaly/issues/528 and https://gitlab.com/gitlab-org/gitaly/issues/529
Related to https://gitlab.com/gitlab-org/gitlab-workhorse/merge_requests/200
## What does this MR do?
## Are ther...~Conversation: https://gitlab.com/gitlab-org/gitaly/issues/528 and https://gitlab.com/gitlab-org/gitaly/issues/529
Related to https://gitlab.com/gitlab-org/gitlab-workhorse/merge_requests/200
## What does this MR do?
## Are there points in the code the reviewer needs to double check?
## Why was this MR needed?
## Screenshots (if relevant)
## Does this MR meet the acceptance criteria?
- [ ] [Changelog entry](https://docs.gitlab.com/ee/development/changelog.html) added, if necessary
- [ ] [Documentation created/updated](https://docs.gitlab.com/ee/development/doc_styleguide.html)
- [ ] API support added
- [ ] Tests added for this feature/bug
- Review
- [ ] Has been reviewed by UX
- [ ] Has been reviewed by Frontend
- [ ] Has been reviewed by Backend
- [ ] Has been reviewed by Database
- [ ] Conform by the [merge request performance guides](https://docs.gitlab.com/ee/development/merge_request_performance_guidelines.html)
- [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
## What are the relevant issue numbers?10.1username-removed-506061username-removed-506061https://staging.gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/14713Pass git object dir attributes as relative paths to Gitaly2017-10-25T00:07:24Zusername-removed-506061Pass git object dir attributes as relative paths to Gitaly## What does this MR do?
Reference https://gitlab.com/gitlab-org/gitaly/issues/629
## Are there points in the code the reviewer needs to double check?
## Why was this MR needed?
## Screenshots (if relevant)
## Does this MR...## What does this MR do?
Reference https://gitlab.com/gitlab-org/gitaly/issues/629
## Are there points in the code the reviewer needs to double check?
## Why was this MR needed?
## Screenshots (if relevant)
## Does this MR meet the acceptance criteria?
- [ ] [Changelog entry](https://docs.gitlab.com/ee/development/changelog.html) added, if necessary
- [ ] [Documentation created/updated](https://docs.gitlab.com/ee/development/doc_styleguide.html)
- [ ] API support added
- [ ] Tests added for this feature/bug
- Review
- [ ] Has been reviewed by UX
- [ ] Has been reviewed by Frontend
- [ ] Has been reviewed by Backend
- [ ] Has been reviewed by Database
- [ ] Conform by the [merge request performance guides](https://docs.gitlab.com/ee/development/merge_request_performance_guidelines.html)
- [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
## What are the relevant issue numbers?10.1username-removed-506061username-removed-506061https://staging.gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/14753WIP: Use Gitaly's RepositoryService.HasLocalBranches RPC2018-12-19T12:27:11Zusername-removed-367626WIP: Use Gitaly's RepositoryService.HasLocalBranches RPC## What does this MR do?
Use Gitaly's RepositoryService.HasLocalBranches RPC
## Are there points in the code the reviewer needs to double check?
No
## Why was this MR needed?
https://gitlab.com/gitlab-org/gitaly/issues/630
## Does ...## What does this MR do?
Use Gitaly's RepositoryService.HasLocalBranches RPC
## Are there points in the code the reviewer needs to double check?
No
## Why was this MR needed?
https://gitlab.com/gitlab-org/gitaly/issues/630
## Does this MR meet the acceptance criteria?
- [ ] [Changelog entry](https://docs.gitlab.com/ee/development/changelog.html) added, if necessary
- [ ] [Documentation created/updated](https://docs.gitlab.com/ee/development/doc_styleguide.html)
- [ ] API support added
- [ ] Tests added for this feature/bug
- Review
- [ ] Has been reviewed by UX
- [ ] Has been reviewed by Frontend
- [ ] Has been reviewed by Backend
- [ ] Has been reviewed by Database
- [ ] Conform by the [merge request performance guides](https://docs.gitlab.com/ee/development/merge_request_performance_guidelines.html)
- [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
## What are the relevant issue numbers?
https://gitlab.com/gitlab-org/gitaly/issues/63010.1James RamsayJames Ramsay