Skip to content
GitLab Next
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • gitaly gitaly
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 565
    • Issues 565
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 58
    • Merge requests 58
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GitLab.org
  • gitalygitaly
  • Merge requests
  • !4224

Increase upload-pack copy buffer sizes

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Jacob Vosmaer requested to merge jv-upload-pack-buffer-size into master Jan 04, 2022
  • Overview 6
  • Commits 2
  • Pipelines 2
  • Changes 4

This changes the upload-pack response copy buffer size in both Git and Gitaly. This is an efficiency improvement for Git fetch traffic. It only makes a difference when the pack-objects cache is enabled; without that it is performance-neutral.

Also see gitlab-git!12 (closed).

I did an experiment where I cloned gitlab-org/gitlab from a local VM, while running perf record at 99Hz. This is with the pack-objects cache enabled, and it's a cache hit. What is nice that we not only save CPU cycles in git and gitaly (the binaries touched by this MR) but also in praefect and gitlab-workhorse.

Process CPU samples before CPU samples after
git 326 179
gitaly 444 347
praefect 480 333
gitlab-workhorse 442 367
Edited Jan 04, 2022 by Jacob Vosmaer
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: jv-upload-pack-buffer-size