##// END OF EJS Templates
branchmap-v3: introduce a "stop_rev" argument to `headsrevs`...
branchmap-v3: introduce a "stop_rev" argument to `headsrevs` The `headsrevs` method of the revlog already have a `revs` argument to compute the headrevs of a limited set of heads. However, it disable the use of the native compiled code to compute the heads, which slows down the branchmap v3 code a lot. The branchmap v3 usage is actually quite constrained as we will always only ignores a part at the top of the graph. So we could be significantly faster. We start by making small change to the python side to improve the situation and introduce the new API. More collaboration with the native code are coming later. This massively speedup operation and close most of the remaining gaps between branchmap-v3 and branchmap-v2. especially on repository with many revs like mozilla-try. A small overhead remains mostly because the `headrevs` logic currently has some inefficiently. We will look into them from there. ### benchmark.name = hg.command.unbundle # bin-env-vars.hg.py-re2-module = default # benchmark.variants.issue6528 = disabled # benchmark.variants.resource-usage = default # benchmark.variants.reuse-external-delta-parent = yes # benchmark.variants.revs = any-1-extra-rev # benchmark.variants.source = unbundle # benchmark.variants.validate = default # benchmark.variants.verbosity = quiet ## data-env-vars.name = netbeans-2018-08-01-zstd-sparse-revlog # bin-env-vars.hg.flavor = default branch-v2: 0.233711 ~~~~~ branch-v3 before: 0.368769 (+57.79%, +0.14) branch-v3 after: 0.239857 (+2.63%, +0.01) # bin-env-vars.hg.flavor = rust branch-v2: 0.235230 ~~~~~ branch-v3 before: 0.372460 (+58.34%, +0.14) branch-v3 after: 0.240972 (+2.44%, +0.01) ## data-env-vars.name = netbeans-2018-08-01-ds2-pnm # bin-env-vars.hg.flavor = rust branch-v2: 0.255586 ~~~~~ branch-v3 before: 0.318907 (+24.78%, +0.06) branch-v3 after: 0.268560 (+5.08%, +0.01) ## data-env-vars.name = mozilla-central-2024-03-22-zstd-sparse-revlog # bin-env-vars.hg.flavor = default branch-v2: 0.339010 ~~~~~ branch-v3 before: 0.349752 (+3.17%, +0.01) branch-v3 after: 0.349389 (+3.06%, +0.01) # bin-env-vars.hg.flavor = rust branch-v2: 0.346525 ~~~~~ branch-v3 before: 0.354300 (+2.24%, +0.01) branch-v3 after: 0.355661 (+2.64%, +0.01) ## data-env-vars.name = mozilla-central-2024-03-22-ds2-pnm # bin-env-vars.hg.flavor = rust branch-v2: 0.380202 ~~~~~ branch-v3 before: 0.396293 (+4.23%, +0.02) branch-v3 after: 0.408851 (+7.54%, +0.03) ## data-env-vars.name = mozilla-unified-2024-03-22-zstd-sparse-revlog # bin-env-vars.hg.flavor = default branch-v2: 0.412165 ~~~~~ branch-v3 before: 0.424769 (+3.06%, +0.01) branch-v3 after: 0.427782 (+3.79%, +0.02) # bin-env-vars.hg.flavor = rust branch-v2: 0.412397 ~~~~~ branch-v3 before: 0.421796 (+2.28%, +0.01) branch-v3 after: 0.422354 (+2.41%, +0.01) ## data-env-vars.name = mozilla-unified-2024-03-22-ds2-pnm # bin-env-vars.hg.flavor = rust branch-v2: 0.429501 ~~~~~ branch-v3 before: 0.443849 (+3.34%, +0.01) branch-v3 after: 0.443197 (+3.19%, +0.01) ## data-env-vars.name = mozilla-try-2024-03-26-zstd-sparse-revlog # bin-env-vars.hg.flavor = default branch-v2: 3.403171 ~~~~~ branch-v3 before: 6.234055 (+83.18%, +2.83) branch-v3 after: 3.819477 (+12.23%, +0.42) # bin-env-vars.hg.flavor = rust branch-v2: 3.454876 ~~~~~ branch-v3 before: 6.307813 (+82.58%, +2.85) branch-v3 after: 3.590284 (+3.92%, +0.14) ## data-env-vars.name = mozilla-try-2024-03-26-ds2-pnm # bin-env-vars.hg.flavor = rust branch-v2: 3.465435 ~~~~~ branch-v3 before: 5.176076 (+49.36%, +1.71) branch-v3 after: 3.633278 (+4.84%, +0.17)

File last commit:

r52756:f4733654 default
r52870:42a116f1 default
Show More
pvec.py
227 lines | 5.9 KiB | text/x-python | PythonLexer
Matt Mackall
pvec: introduce pvecs
r16249 # pvec.py - probabilistic vector clocks for Mercurial
#
Raphaël Gomès
contributor: change mentions of mpm to olivia...
r47575 # Copyright 2012 Olivia Mackall <olivia@selenic.com>
Matt Mackall
pvec: introduce pvecs
r16249 #
# This software may be used and distributed according to the terms of the
# GNU General Public License version 2 or any later version.
'''
A "pvec" is a changeset property based on the theory of vector clocks
that can be compared to discover relatedness without consulting a
graph. This can be useful for tasks like determining how a
disconnected patch relates to a repository.
Currently a pvec consist of 448 bits, of which 24 are 'depth' and the
remainder are a bit vector. It is represented as a 70-character base85
string.
Construction:
- a root changeset has a depth of 0 and a bit vector based on its hash
- a normal commit has a changeset where depth is increased by one and
one bit vector bit is flipped based on its hash
- a merge changeset pvec is constructed by copying changes from one pvec into
the other to balance its depth
Properties:
- for linear changes, difference in depth is always <= hamming distance
- otherwise, changes are probably divergent
- when hamming distance is < 200, we can reliably detect when pvecs are near
Issues:
- hamming distance ceases to work over distances of ~ 200
- detecting divergence is less accurate when the common ancestor is very close
to either revision or total distance is high
- this could probably be improved by modeling the relation between
delta and hdist
Uses:
- a patch pvec can be used to locate the nearest available common ancestor for
resolving conflicts
- ordering of patches can be established without a DAG
- two head pvecs can be compared to determine whether push/pull/merge is needed
and approximately how many changesets are involved
- can be used to find a heuristic divergence measure between changesets on
different branches
'''
Matt Harbison
typing: add `from __future__ import annotations` to most files...
r52756 from __future__ import annotations
Gregory Szorc
pvec: use absolute_import
r27501
from .node import nullrev
from . import (
Gregory Szorc
global: use pycompat.xrange()...
r38806 pycompat,
Gregory Szorc
pvec: use absolute_import
r27501 util,
)
Matt Mackall
pvec: introduce pvecs
r16249
Augie Fackler
formatting: blacken the codebase...
r43346 _size = 448 # 70 chars b85-encoded
Augie Fackler
pvec: migrate to modern integer division...
r43766 _bytes = _size // 8
Matt Mackall
pvec: introduce pvecs
r16249 _depthbits = 24
Augie Fackler
pvec: migrate to modern integer division...
r43766 _depthbytes = _depthbits // 8
Matt Mackall
pvec: introduce pvecs
r16249 _vecbytes = _bytes - _depthbytes
_vecbits = _vecbytes * 8
Augie Fackler
pvec: migrate to modern integer division...
r43766 _radius = (_vecbits - 30) // 2 # high probability vectors are related
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Mackall
pvec: introduce pvecs
r16249
def _bin(bs):
'''convert a bytestring to a long'''
v = 0
for b in bs:
v = v * 256 + ord(b)
return v
Augie Fackler
formatting: blacken the codebase...
r43346
pytype: move some type comment to proper annotation...
r52180 def _str(v: int, l: int) -> bytes:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 bs = b""
Manuel Jacob
py3: replace `pycompat.xrange` by `range`
r50179 for p in range(l):
Augie Fackler
pvec: fix overlooked chr() call...
r43764 bs = pycompat.bytechr(v & 255) + bs
Matt Mackall
pvec: introduce pvecs
r16249 v >>= 8
return bs
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Mackall
pvec: introduce pvecs
r16249 def _split(b):
'''depth and bitvec'''
return _bin(b[:_depthbytes]), _bin(b[_depthbytes:])
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Mackall
pvec: introduce pvecs
r16249 def _join(depth, bitvec):
return _str(depth, _depthbytes) + _str(bitvec, _vecbytes)
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Mackall
pvec: introduce pvecs
r16249 def _hweight(x):
c = 0
while x:
if x & 1:
c += 1
x >>= 1
return c
Augie Fackler
formatting: blacken the codebase...
r43346
Manuel Jacob
py3: replace `pycompat.xrange` by `range`
r50179 _htab = [_hweight(x) for x in range(256)]
Matt Mackall
pvec: introduce pvecs
r16249
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Mackall
pvec: introduce pvecs
r16249 def _hamming(a, b):
'''find the hamming distance between two longs'''
d = a ^ b
c = 0
while d:
Augie Fackler
formatting: blacken the codebase...
r43346 c += _htab[d & 0xFF]
Matt Mackall
pvec: introduce pvecs
r16249 d >>= 8
return c
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Mackall
pvec: introduce pvecs
r16249 def _mergevec(x, y, c):
# Ideally, this function would be x ^ y ^ ancestor, but finding
# ancestors is a nuisance. So instead we find the minimal number
# of changes to balance the depth and hamming distance
d1, v1 = x
d2, v2 = y
if d1 < d2:
d1, d2, v1, v2 = d2, d1, v2, v1
hdist = _hamming(v1, v2)
ddist = d1 - d2
v = v1
Augie Fackler
formatting: blacken the codebase...
r43346 m = v1 ^ v2 # mask of different bits
Matt Mackall
pvec: introduce pvecs
r16249 i = 1
if hdist > ddist:
# if delta = 10 and hdist = 100, then we need to go up 55 steps
# to the ancestor and down 45
Augie Fackler
pvec: migrate to modern integer division...
r43766 changes = (hdist - ddist + 1) // 2
Matt Mackall
pvec: introduce pvecs
r16249 else:
# must make at least one change
changes = 1
depth = d1 + changes
# copy changes from v2
if m:
while changes:
if m & i:
v ^= i
changes -= 1
i <<= 1
else:
v = _flipbit(v, c)
return depth, v
Augie Fackler
formatting: blacken the codebase...
r43346
Matt Mackall
pvec: introduce pvecs
r16249 def _flipbit(v, node):
# converting bit strings to longs is slow
Augie Fackler
formatting: blacken the codebase...
r43346 bit = (hash(node) & 0xFFFFFFFF) % _vecbits
return v ^ (1 << bit)
Matt Mackall
pvec: introduce pvecs
r16249
def ctxpvec(ctx):
'''construct a pvec for ctx while filling in the cache'''
Matt Harbison
pvec: replace 'ctx._repo' with 'ctx.repo()'
r24339 r = ctx.repo()
safehasattr: drop usage in favor of hasattr...
r51821 if not hasattr(r, "_pveccache"):
Matt Mackall
pvec: introduce pvecs
r16249 r._pveccache = {}
pvc = r._pveccache
if ctx.rev() not in pvc:
cl = r.changelog
Manuel Jacob
py3: replace `pycompat.xrange` by `range`
r50179 for n in range(ctx.rev() + 1):
Matt Mackall
pvec: introduce pvecs
r16249 if n not in pvc:
node = cl.node(n)
p1, p2 = cl.parentrevs(n)
if p1 == nullrev:
# start with a 'random' vector at root
pvc[n] = (0, _bin((node * 3)[:_vecbytes]))
elif p2 == nullrev:
d, v = pvc[p1]
pvc[n] = (d + 1, _flipbit(v, node))
else:
pvc[n] = _mergevec(pvc[p1], pvc[p2], node)
bs = _join(*pvc[ctx.rev()])
Yuya Nishihara
base85: proxy through util module...
r32200 return pvec(util.b85encode(bs))
Matt Mackall
pvec: introduce pvecs
r16249
Augie Fackler
formatting: blacken the codebase...
r43346
Gregory Szorc
py3: use class X: instead of class X(object):...
r49801 class pvec:
Matt Mackall
pvec: introduce pvecs
r16249 def __init__(self, hashorctx):
Matt Harbison
pvec: fix a `str` type conditional for py3...
r44184 if isinstance(hashorctx, bytes):
Matt Mackall
pvec: introduce pvecs
r16249 self._bs = hashorctx
Yuya Nishihara
base85: proxy through util module...
r32200 self._depth, self._vec = _split(util.b85decode(hashorctx))
Matt Mackall
pvec: introduce pvecs
r16249 else:
Bryan O'Sullivan
pvec: use the correct name for an identifier...
r18918 self._vec = ctxpvec(hashorctx)
Matt Mackall
pvec: introduce pvecs
r16249
def __str__(self):
return self._bs
def __eq__(self, b):
return self._vec == b._vec and self._depth == b._depth
def __lt__(self, b):
delta = b._depth - self._depth
if delta < 0:
Augie Fackler
formatting: blacken the codebase...
r43346 return False # always correct
Matt Mackall
pvec: introduce pvecs
r16249 if _hamming(self._vec, b._vec) > delta:
return False
return True
def __gt__(self, b):
return b < self
def __or__(self, b):
delta = abs(b._depth - self._depth)
if _hamming(self._vec, b._vec) <= delta:
return False
return True
def __sub__(self, b):
if self | b:
Augie Fackler
formatting: byteify all mercurial/ and hgext/ string literals...
r43347 raise ValueError(b"concurrent pvecs")
Matt Mackall
pvec: introduce pvecs
r16249 return self._depth - b._depth
def distance(self, b):
d = abs(b._depth - self._depth)
h = _hamming(self._vec, b._vec)
return max(d, h)
def near(self, b):
dist = abs(b.depth - self._depth)
if dist > _radius or _hamming(self._vec, b._vec) > _radius:
return False