Skip to content

Conversation

@nicolaiskogheim
Copy link
Collaborator

Speedup is gained by two main edits

  • run effort in parallel
  • remove one call to git log (we can use one for both statistics)

This PR will conflict with #390 and/or #391, so I will do a rebase if those are merged.

I have tested this with the Hugo blog engine, with 24 seconds on git effort vs. 7 seconds with this patch. (Those numbers are obviously to be taken with a grain of salt) This was on my computer with two physical cores.

@hemanth
Copy link
Collaborator

hemanth commented Jul 10, 2015

Thank you.

hemanth added a commit that referenced this pull request Jul 10, 2015
@hemanth hemanth merged commit fa3cf6a into tj:master Jul 10, 2015
@nicolaiskogheim
Copy link
Collaborator Author

Please don't merge in my WIP-branch from my RFC pull request next time without letting me know first.
Thank you.

@hemanth
Copy link
Collaborator

hemanth commented Jul 13, 2015

Sorry, this pull request couldn't be reverted automatically. It may have already been reverted, or the content may have changed since it was merged.

Reverting will result in merge conflict, can you please do a new PR, sorry for the inconvenience.

@nicolaiskogheim
Copy link
Collaborator Author

It is working as it is now, but I will open a new PR with some small touch ups. I guess it was a little late when I wrote that comment, didn't mean to sound harsh.

@hemanth
Copy link
Collaborator

hemanth commented Jul 13, 2015

👍

@nicolaiskogheim nicolaiskogheim deleted the WIP-effort-parallelize branch July 26, 2015 15:09
@qw3rtman qw3rtman mentioned this pull request Dec 27, 2015
tardypad pushed a commit to tardypad/git-extras that referenced this pull request Jul 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants