commit-graph: Introduce `git_commit_list_generation_cmp` by lhchavez · Pull Request #5766 · libgit2/libgit2 · GitHub
Skip to content

commit-graph: Introduce git_commit_list_generation_cmp#5766

Merged
ethomson merged 1 commit into
libgit2:mainfrom
lhchavez:cgraph-generation-cmp
Jul 27, 2021
Merged

commit-graph: Introduce git_commit_list_generation_cmp#5766
ethomson merged 1 commit into
libgit2:mainfrom
lhchavez:cgraph-generation-cmp

Conversation

@lhchavez

@lhchavez lhchavez commented Jan 7, 2021

Copy link
Copy Markdown
Contributor

This change makes calculations of merge-bases a bit faster when there
are complex graphs and the commit times cause visiting nodes multiple
times. This is done by visiting the nodes in the graph in reverse
generation order when the generation number is available instead of
commit timestamp. If the generation number is missing in any pair of
commits, it can safely fall back to the old heuristic with no negative
side-effects.

Part of: #5757

@lhchavez lhchavez mentioned this pull request Jan 7, 2021
8 tasks
@lhchavez lhchavez force-pushed the cgraph-generation-cmp branch 2 times, most recently from fc244b7 to 29d2d95 Compare January 7, 2021 04:54
Base automatically changed from master to main January 7, 2021 10:10
@lhchavez lhchavez force-pushed the cgraph-generation-cmp branch 3 times, most recently from 46c29f8 to 411e945 Compare January 10, 2021 21:28
@lhchavez lhchavez force-pushed the cgraph-generation-cmp branch from 411e945 to cf9f047 Compare March 11, 2021 02:11
This change makes calculations of merge-bases a bit faster when there
are complex graphs and the commit times cause visiting nodes multiple
times. This is done by visiting the nodes in the graph in reverse
generation order when the generation number is available instead of
commit timestamp. If the generation number is missing in any pair of
commits, it can safely fall back to the old heuristic with no negative
side-effects.

Part of: libgit2#5757
@lhchavez lhchavez force-pushed the cgraph-generation-cmp branch from cf9f047 to 6f54414 Compare July 27, 2021 01:46
@ethomson ethomson merged commit 08c7912 into libgit2:main Jul 27, 2021
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