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

Conversation

@marcinwyszynski
Copy link
Contributor

Brilliant piece of Go code, congrats! Out of curiosity - have you tried benchmarking this against Go libgit2 bindings running from ramdisk? For a small side project where I needed git blame functionality I got 10-20x speed boost from doing just that.

Brilliant piece of Go code, congrats! Out of curiosity - have you tried benchmarking this against Golang's libgit2 bindings running from ramdisk? For a small side project where I needed `git blame` functionality I got 10-20x speed boost from doing just that.
@codecov-io
Copy link

Current coverage is 61.48%

Merging #21 into master will not affect coverage as of bf10db4

@@            master     #21   diff @@
======================================
  Files           24      24       
  Stmts         1532    1532       
  Branches       203     203       
  Methods          0       0       
======================================
  Hit            942     942       
  Partial         92      92       
  Missed         498     498       

Review entire Coverage Diff as of bf10db4

Powered by Codecov. Updated on successful CI builds.

mcuadros added a commit that referenced this pull request Feb 16, 2016
@mcuadros mcuadros merged commit 0b2065d into src-d:master Feb 16, 2016
@mcuadros
Copy link
Contributor

Hi Marcin,

Thanks for the badge, we are working in a proper README right now. 💃

About git2go in ramdisk vs go-git, we use go-git with some tweaks, for example we discard every binary file when we are reading the packfile so this is much faster than git2go, I don't have the numbers here but we compared it some months ago.

BTW We are working also in a ObjectStorage implementation that analyzes the files and discarded the content making the memory foot print very low.

About the git-blame on ramdisk, our current implementation of blame is in early stages so we cannot compare it. But yes maybe will be faster run the libgit2 version.

Obviously a ramdisk is an easy solution for this problems, but dealing with big repositories becomes a memory problem if you don't discard the binaries.

@marcinwyszynski
Copy link
Contributor Author

Thanks!

borat-dancing

@marcinwyszynski marcinwyszynski deleted the patch-1 branch February 16, 2016 10:34
mcuadros added a commit that referenced this pull request Jan 31, 2017
gsalingu-ovhus pushed a commit to gsalingu-ovhus/go-git that referenced this pull request Mar 28, 2019
Remove ovhto shorten backend
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.

3 participants