What's the difference between `git diff --patience` and `git diff --histogram`?
This histogram strategy was introduced in git 1.7.7 (Sept 2011), with the following description (as mentioned by the OP)
"
git diff
" learned a "--histogram
" option to use a different diff generation machinery stolen from jgit, which might give better performance.
JGit includes src/org/eclipse/jgit/diff/HistogramDiff.java
and tst/org/eclipse/jgit/diff/HistogramDiffTest.java
The description there is fairly complete:
HistogramDiff
An extended form of Bram Cohen's patience diff algorithm.
This implementation was derived by using the 4 rules that are outlined in Bram Cohen's blog, and then was further extended to support low-occurrence common elements.
The basic idea of the algorithm is to create a histogram of occurrences for each element of sequence A. Each element of sequence B is then considered in turn. If the element also exists in sequence A, and has a lower occurrence count, the positions are considered as a candidate for the longest common subsequence (LCS).
After scanning of B is complete the LCS that has the lowest number of occurrences is chosen as a split point. The region is split around the LCS, and the algorithm is recursively applied to the sections before and after the LCS.By always selecting a LCS position with the lowest occurrence count, this algorithm behaves exactly like Bram Cohen's patience diff whenever there is a unique common element available between the two sequences.
When no unique elements exist, the lowest occurrence element is chosen instead.
This offers more readable diffs than simply falling back on the standard Myers'O(ND)
algorithm would produce.To prevent the algorithm from having an
O(N^2)
running time, an upper limit on the number of unique elements in a histogram bucket is configured by#setMaxChainLength(int)
.
If sequence A has more than this many elements that hash into the same hash bucket, the algorithm passes the region to#setFallbackAlgorithm(DiffAlgorithm)
.
If no fallback algorithm is configured, the region is emitted as a replace edit.During scanning of sequence B, any element of A that occurs more than
#setMaxChainLength(int)
times is never considered for an LCS match position, even if it is common between the two sequences. This limits the number of locations in sequence A that must be considered to find the LCS,and helps maintain a lower running time bound.So long as
#setMaxChainLength(int)
is a small constant (such as 64), the algorithm runs inO(N * D)
time, whereN
is the sum of the input lengths andD
is the number of edits in the resultingEditList
.
If the suppliedSequenceComparator
has a good hash function, this implementation typically out-performsMyersDiff
, even though its theoretical running time is the same.This implementation has an internal limitation that prevents it from handling sequences with more than 268,435,456 (2^28) elements
Note that this kind of algo was already used for pack_check, back in 2006 (git 1.3), for git-verify-pack -v
. It was reused for index-pack in git 1.7.7
Commit 8c912ee actually introduced --histogram
to diff:
Port JGit's HistogramDiff algorithm over to C. Rough numbers (TODO) show that it is faster than its
--patience
cousin, as well as the default Meyers algorithm.The implementation has been reworked to use structs and pointers, instead of bitmasks, thus doing away with JGit's
2^28
line limit.We also use
xdiff
's default hash table implementation (xdl_hash_bits()
withXDL_HASHLONG()
) for convenience.
commit 8555123 (git 1.7.10, April 2012) added:
8c912ee (teach
--histogram
todiff
, 2011-07-12) claimed histogram diff was faster than both Myers and patience.We have since incorporated a performance testing framework, so add a test that compares the various diff tasks performed in a real '
log -p
' workload.
This does indeed show that histogram diff slightly beats Myers, while patience is much slower than the others.
Finally, commit 07ab4de (git 1.8.2, March 2013) add
config: Introduce diff.algorithm variable
Some users or projects prefer different algorithms over others, e.g. patience over myers or similar.
However, specifying appropriate argument every time diff is to be used is impractical. Moreover, creating an alias doesn't play nicely with other tools based on diff (git-show
for instance).Hence, a configuration variable which is able to set specific algorithm is needed.
For now, these four values are accepted:
- '
myers
' (which has the same effect as not setting the config variable at all),- '
minimal
',- '
patience
' and- '
histogram
'.
Commit 07924d4 added concurrently the --diff-algorithm
command line option.
As the OP Stuart P. Bentley mentions in the comments:
you can configure Git to use histogram by default with:
git config --global diff.algorithm histogram
Update: Git 2.12 (Q1 2017) will retire the "fast hash" that had disastrous performance issues in some corner cases.
See commit 1f7c926 (01 Dec 2016) by Jeff King (peff
).
(Merged by Junio C Hamano -- gitster
-- in commit 731490b, 19 Dec 2016)
xdiff
: dropXDL_FAST_HASH
The
xdiff
code hashes every line of both sides of a diff, and then compares those hashes to find duplicates. The overall performance depends both on how fast we can compute the hashes, but also on how many hash collisions we see.The idea of
XDL_FAST_HASH
is to speed up the hash computation.
But the generated hashes have worse collision behavior. This means that in some cases it speeds diffs up (running "git log -p
" ongit.git
improves by~8%
with it), but in others it can slow things down. One pathological case saw over a 100x slowdown.There may be a better hash function that covers both properties, but in the meantime we are better off with the original hash. It's slightly slower in the common case, but it has fewer surprising pathological cases.
Note: "git diff --histogram
" had a bad memory usage pattern, which has
been rearranged to reduce the peak usage, with Git 2.19 (Q3 2018).
See commit 79cb2eb, commit 64c4e8b, commit c671d4b, commit 2820985 (19 Jul 2018) by Stefan Beller (stefanbeller
).
(Merged by Junio C Hamano -- gitster
-- in commit 57fbd8e, 15 Aug 2018)
xdiff/xhistogram
: move index allocation intofind_lcs
This fixes a memory issue when recursing a lot, which can be reproduced as
seq 1 100000 >one seq 1 4 100000 >two git diff --no-index --histogram one two
Before this patch,
histogram_diff
would call itself recursively before callingfree_index
, which would mean a lot of memory is allocated during the recursion and only freed afterwards.By moving the memory allocation (and its free call) into
find_lcs
, the memory is free'd before we recurse, such that memory is reused in the next step of the recursion instead of using new memory.This addresses only the memory pressure, not the run time complexity, that is also awful for the corner case outlined above.