Skip to content

commit-graph: avoid unnecessary tag deference when merging

Taylor Blau git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest1 patch 29 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/1commit-graph.c: avoid unnecessary tag dereference when mergingcommit-graph.c+6 -226 replies

Reviews trailers per patch

The patch has no Reviewed-by or Acked-by.

PatchJeff King
1/1Co-authored-by

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/1] commit-graph: avoid unnecessary tag deference when merging

Hi,

Here's a patch that Peff and I were kicking around from a couple of months ago when we were investigating deploying commit-graphs at GitHub. Particularly, we were looking into why merging two commit-graphs after a write with '--split' took as much time as we were seeing in our infrastructure.

This patch avoids an unnecessary tag dereference in 'merge_commit_graph()', which can improve the running time of a commit-graph write by around ~7.4% on average.

Thanks in advance for your review.

Taylor Blau (1):
commit-graph.c: avoid unnecessary tag dereference when merging

 commit-graph.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

--
2.26.0.rc2.311.g8e52d2684b