diff options
| author | Jeff King <peff@peff.net> | 2019-09-07 01:04:40 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2019-09-09 10:56:50 -0700 |
| commit | dd2e50a84ea431a6cec69f37251f29bf3cfcbb68 (patch) | |
| tree | 665279f4126028df11932b2e584ed21c26ebe629 | |
| parent | 67fa6aac5af508b307c0d6968a28d50d14e7c026 (diff) | |
| download | git-dd2e50a84ea431a6cec69f37251f29bf3cfcbb68.tar.gz | |
commit-graph: turn off save_commit_buffer
The commit-graph tool may read a lot of commits, but it only cares about
parsing their metadata (parents, trees, etc) and doesn't ever show the
messages to the user. And so it should not need save_commit_buffer,
which is meant for holding onto the object data of parsed commits so
that we can show them later. In fact, it's quite harmful to do so.
According to massif, the max heap of "git commit-graph write
--reachable" in linux.git before/after this patch (removing the commit
graph file in between) goes from ~1.1GB to ~270MB.
Which isn't surprising, since the difference is about the sum of the
uncompressed sizes of all commits in the repository, and this was
equivalent to leaking them.
This obviously helps if you're under memory pressure, but even without
it, things go faster. My before/after times for that command (without
massif) went from 12.521s to 11.874s, a speedup of ~5%.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin/commit-graph.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/builtin/commit-graph.c b/builtin/commit-graph.c index 38027b83d9..3b5d58a5fa 100644 --- a/builtin/commit-graph.c +++ b/builtin/commit-graph.c @@ -249,6 +249,8 @@ int cmd_commit_graph(int argc, const char **argv, const char *prefix) builtin_commit_graph_usage, PARSE_OPT_STOP_AT_NON_OPTION); + save_commit_buffer = 0; + if (argc > 0) { if (!strcmp(argv[0], "read")) return graph_read(argc, argv); |