Skip to content

Commit 6ef3487

Browse files
derrickstoleegitster
authored andcommitted
commit-graph: remove Future Work section
The commit-graph feature began with a long list of planned benefits, most of which are now complete. The future work section has only a few items left. As for making more algorithms aware of generation numbers, some are only waiting for generation number v2 to ensure the performance matches the existing behavior using commit date. It is unlikely that we will ever send a commit-graph file as part of the protocol, since we would need to verify the data, and that is expensive. If we want to start trusting remote content, then that item can be investigated again. While there is more work to be done on the feature, having a section of the docs devoted to a TODO list is wasteful and hard to keep up-to-date. Signed-off-by: Derrick Stolee <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 544b58b commit 6ef3487

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

Documentation/technical/commit-graph.txt

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -127,23 +127,6 @@ Design Details
127127
helpful for these clones, anyway. The commit-graph will not be read or
128128
written when shallow commits are present.
129129

130-
Future Work
131-
-----------
132-
133-
- After computing and storing generation numbers, we must make graph
134-
walks aware of generation numbers to gain the performance benefits they
135-
enable. This will mostly be accomplished by swapping a commit-date-ordered
136-
priority queue with one ordered by generation number. The following
137-
operations are important candidates:
138-
139-
- 'log --topo-order'
140-
- 'tag --merged'
141-
142-
- A server could provide a commit-graph file as part of the network protocol
143-
to avoid extra calculations by clients. This feature is only of benefit if
144-
the user is willing to trust the file, because verifying the file is correct
145-
is as hard as computing it from scratch.
146-
147130
Related Links
148131
-------------
149132
[0] https://bugs.chromium.org/p/git/issues/detail?id=8

0 commit comments

Comments
 (0)