Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

plumbing, storage: add bases to the common cache #929

Merged
merged 1 commit into from
Aug 22, 2018

Conversation

jfontan
Copy link
Contributor

@jfontan jfontan commented Aug 22, 2018

After clone only resolved deltas were added to the cache. This caused slowdowns accessing objects in small repositories.

It also makes packfiles reuse delta cache from the store. Previously it created a new delta cache each time a packfile object was created. This also slowed down a bit accessing objects and had an impact on memory consumption when bases are added to the cache.

Times to clone and push changes using borges:

repo 4.5.0 4.6.0 now
octoprint-tft 4.5s 8s 4.4s
upsilon 0:54 1:38 0:54
numpy 3:03 3:29 3:06
tensorflow 12:59 14:48 12:38
bismuth 14:18 7:31 7:05

After clone only resolved deltas were added to the cache. This caused
slowdowns in small repositories where most objects can be held in cache.

It also makes packfiles reuse delta cache from the store. Previously it
created a new delta cache each time a packfile object was created. This
also slowed down a bit accessing objects and had an impact on memory
consumption when bases are added to the cache.

Signed-off-by: Javi Fontan <[email protected]>
@mcuadros mcuadros merged commit 005d5dc into src-d:master Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants