~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/chk_map.py

  • Committer: Vincent Ladeuil
  • Date: 2017-01-17 13:48:10 UTC
  • mfrom: (6615.3.6 merges)
  • mto: This revision was merged to the branch mainline in revision 6620.
  • Revision ID: v.ladeuil+lp@free.fr-20170117134810-j9p3lidfy6pfyfsc
Merge 2.7, resolving conflicts

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2008, 2009 Canonical Ltd
 
1
# Copyright (C) 2008-2011 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
37
37
 
38
38
"""
39
39
 
 
40
from __future__ import absolute_import
 
41
 
40
42
import heapq
 
43
import threading
41
44
 
42
45
from bzrlib import lazy_import
43
46
lazy_import.lazy_import(globals(), """
44
47
from bzrlib import (
45
48
    errors,
46
 
    versionedfile,
47
49
    )
48
50
""")
49
51
from bzrlib import (
 
52
    errors,
50
53
    lru_cache,
51
54
    osutils,
52
55
    registry,
 
56
    static_tuple,
53
57
    trace,
54
58
    )
 
59
from bzrlib.static_tuple import StaticTuple
55
60
 
56
61
# approx 4MB
57
62
# If each line is 50 bytes, and you have 255 internal pages, with 255-way fan
58
63
# out, it takes 3.1MB to cache the layer.
59
64
_PAGE_CACHE_SIZE = 4*1024*1024
60
 
# We are caching bytes so len(value) is perfectly accurate
61
 
_page_cache = lru_cache.LRUSizeCache(_PAGE_CACHE_SIZE)
 
65
# Per thread caches for 2 reasons:
 
66
# - in the server we may be serving very different content, so we get less
 
67
#   cache thrashing.
 
68
# - we avoid locking on every cache lookup.
 
69
_thread_caches = threading.local()
 
70
# The page cache.
 
71
_thread_caches.page_cache = None
 
72
 
 
73
def _get_cache():
 
74
    """Get the per-thread page cache.
 
75
 
 
76
    We need a function to do this because in a new thread the _thread_caches
 
77
    threading.local object does not have the cache initialized yet.
 
78
    """
 
79
    page_cache = getattr(_thread_caches, 'page_cache', None)
 
80
    if page_cache is None:
 
81
        # We are caching bytes so len(value) is perfectly accurate
 
82
        page_cache = lru_cache.LRUSizeCache(_PAGE_CACHE_SIZE)
 
83
        _thread_caches.page_cache = page_cache
 
84
    return page_cache
 
85
 
 
86
 
 
87
def clear_cache():
 
88
    _get_cache().clear()
 
89
 
62
90
 
63
91
# If a ChildNode falls below this many bytes, we check for a remap
64
92
_INTERESTING_NEW_SIZE = 50
65
93
# If a ChildNode shrinks by more than this amount, we check for a remap
66
94
_INTERESTING_SHRINKAGE_LIMIT = 20
67
 
# If we delete more than this many nodes applying a delta, we check for a remap
68
 
_INTERESTING_DELETES_LIMIT = 5
69
95
 
70
96
 
71
97
def _search_key_plain(key):
80
106
class CHKMap(object):
81
107
    """A persistent map from string to string backed by a CHK store."""
82
108
 
 
109
    __slots__ = ('_store', '_root_node', '_search_key_func')
 
110
 
83
111
    def __init__(self, store, root_key, search_key_func=None):
84
112
        """Create a CHKMap object.
85
113
 
107
135
            into the map; if old_key is not None, then the old mapping
108
136
            of old_key is removed.
109
137
        """
110
 
        delete_count = 0
 
138
        has_deletes = False
111
139
        # Check preconditions first.
112
 
        new_items = set([key for (old, key, value) in delta if key is not None
113
 
            and old is None])
 
140
        as_st = StaticTuple.from_sequence
 
141
        new_items = set([as_st(key) for (old, key, value) in delta
 
142
                         if key is not None and old is None])
114
143
        existing_new = list(self.iteritems(key_filter=new_items))
115
144
        if existing_new:
116
145
            raise errors.InconsistentDeltaDelta(delta,
119
148
        for old, new, value in delta:
120
149
            if old is not None and old != new:
121
150
                self.unmap(old, check_remap=False)
122
 
                delete_count += 1
 
151
                has_deletes = True
123
152
        for old, new, value in delta:
124
153
            if new is not None:
125
154
                self.map(new, value)
126
 
        if delete_count > _INTERESTING_DELETES_LIMIT:
127
 
            trace.mutter("checking remap as %d deletions", delete_count)
 
155
        if has_deletes:
128
156
            self._check_remap()
129
157
        return self._save()
130
158
 
131
159
    def _ensure_root(self):
132
160
        """Ensure that the root node is an object not a key."""
133
 
        if type(self._root_node) is tuple:
 
161
        if type(self._root_node) is StaticTuple:
134
162
            # Demand-load the root
135
163
            self._root_node = self._get_node(self._root_node)
136
164
 
144
172
        :param node: A tuple key or node object.
145
173
        :return: A node object.
146
174
        """
147
 
        if type(node) is tuple:
 
175
        if type(node) is StaticTuple:
148
176
            bytes = self._read_bytes(node)
149
177
            return _deserialise(bytes, node,
150
178
                search_key_func=self._search_key_func)
153
181
 
154
182
    def _read_bytes(self, key):
155
183
        try:
156
 
            return _page_cache[key]
 
184
            return _get_cache()[key]
157
185
        except KeyError:
158
186
            stream = self._store.get_record_stream([key], 'unordered', True)
159
187
            bytes = stream.next().get_bytes_as('fulltext')
160
 
            _page_cache[key] = bytes
 
188
            _get_cache()[key] = bytes
161
189
            return bytes
162
190
 
163
191
    def _dump_tree(self, include_keys=False):
191
219
            for key, value in sorted(node._items.iteritems()):
192
220
                # Don't use prefix nor indent here to line up when used in
193
221
                # tests in conjunction with assertEqualDiff
194
 
                result.append('      %r %r' % (key, value))
 
222
                result.append('      %r %r' % (tuple(key), value))
195
223
        return result
196
224
 
197
225
    @classmethod
215
243
        root_key = klass._create_directly(store, initial_value,
216
244
            maximum_size=maximum_size, key_width=key_width,
217
245
            search_key_func=search_key_func)
 
246
        if type(root_key) is not StaticTuple:
 
247
            raise AssertionError('we got a %s instead of a StaticTuple'
 
248
                                 % (type(root_key),))
218
249
        return root_key
219
250
 
220
251
    @classmethod
235
266
        node = LeafNode(search_key_func=search_key_func)
236
267
        node.set_maximum_size(maximum_size)
237
268
        node._key_width = key_width
238
 
        node._items = dict(initial_value)
 
269
        as_st = StaticTuple.from_sequence
 
270
        node._items = dict([(as_st(key), val) for key, val
 
271
                                               in initial_value.iteritems()])
239
272
        node._raw_size = sum([node._key_value_len(key, value)
240
 
                              for key,value in initial_value.iteritems()])
 
273
                              for key,value in node._items.iteritems()])
241
274
        node._len = len(node._items)
242
275
        node._compute_search_prefix()
243
276
        node._compute_serialised_prefix()
479
512
    def iteritems(self, key_filter=None):
480
513
        """Iterate over the entire CHKMap's contents."""
481
514
        self._ensure_root()
 
515
        if key_filter is not None:
 
516
            as_st = StaticTuple.from_sequence
 
517
            key_filter = [as_st(key) for key in key_filter]
482
518
        return self._root_node.iteritems(self._store, key_filter=key_filter)
483
519
 
484
520
    def key(self):
485
521
        """Return the key for this map."""
486
 
        if type(self._root_node) is tuple:
 
522
        if type(self._root_node) is StaticTuple:
487
523
            return self._root_node
488
524
        else:
489
525
            return self._root_node._key
498
534
        :param key: A key to map.
499
535
        :param value: The value to assign to key.
500
536
        """
 
537
        key = StaticTuple.from_sequence(key)
501
538
        # Need a root object.
502
539
        self._ensure_root()
503
540
        prefix, node_details = self._root_node.map(self._store, key, value)
514
551
    def _node_key(self, node):
515
552
        """Get the key for a node whether it's a tuple or node."""
516
553
        if type(node) is tuple:
 
554
            node = StaticTuple.from_sequence(node)
 
555
        if type(node) is StaticTuple:
517
556
            return node
518
557
        else:
519
558
            return node._key
520
559
 
521
560
    def unmap(self, key, check_remap=True):
522
561
        """remove key from the map."""
 
562
        key = StaticTuple.from_sequence(key)
523
563
        self._ensure_root()
524
564
        if type(self._root_node) is InternalNode:
525
565
            unmapped = self._root_node.unmap(self._store, key,
532
572
        """Check if nodes can be collapsed."""
533
573
        self._ensure_root()
534
574
        if type(self._root_node) is InternalNode:
535
 
            self._root_node._check_remap(self._store)
 
575
            self._root_node = self._root_node._check_remap(self._store)
536
576
 
537
577
    def _save(self):
538
578
        """Save the map completely.
539
579
 
540
580
        :return: The key of the root node.
541
581
        """
542
 
        if type(self._root_node) is tuple:
 
582
        if type(self._root_node) is StaticTuple:
543
583
            # Already saved.
544
584
            return self._root_node
545
585
        keys = list(self._root_node.serialise(self._store))
553
593
        adding the header bytes, and without prefix compression.
554
594
    """
555
595
 
 
596
    __slots__ = ('_key', '_len', '_maximum_size', '_key_width',
 
597
                 '_raw_size', '_items', '_search_prefix', '_search_key_func'
 
598
                )
 
599
 
556
600
    def __init__(self, key_width=1):
557
601
        """Create a node.
558
602
 
647
691
        the key/value pairs.
648
692
    """
649
693
 
 
694
    __slots__ = ('_common_serialised_prefix',)
 
695
 
650
696
    def __init__(self, search_key_func=None):
651
697
        Node.__init__(self)
652
698
        # All of the keys in this leaf node share this common prefix
653
699
        self._common_serialised_prefix = None
654
 
        self._serialise_key = '\x00'.join
655
700
        if search_key_func is None:
656
701
            self._search_key_func = _search_key_plain
657
702
        else:
695
740
        :param bytes: The bytes of the node.
696
741
        :param key: The key that the serialised node has.
697
742
        """
 
743
        key = static_tuple.expect_static_tuple(key)
698
744
        return _deserialise_leaf_node(bytes, key,
699
745
                                      search_key_func=search_key_func)
700
746
 
840
886
                raise AssertionError('%r must be known' % self._search_prefix)
841
887
            return self._search_prefix, [("", self)]
842
888
 
 
889
    _serialise_key = '\x00'.join
 
890
 
843
891
    def serialise(self, store):
844
892
        """Serialise the LeafNode to store.
845
893
 
870
918
            lines.append(serialized[prefix_len:])
871
919
            lines.extend(value_lines)
872
920
        sha1, _, _ = store.add_lines((None,), (), lines)
873
 
        self._key = ("sha1:" + sha1,)
 
921
        self._key = StaticTuple("sha1:" + sha1,).intern()
874
922
        bytes = ''.join(lines)
875
923
        if len(bytes) != self._current_size():
876
924
            raise AssertionError('Invalid _current_size')
877
 
        _page_cache.add(self._key, bytes)
 
925
        _get_cache()[self._key] = bytes
878
926
        return [self._key]
879
927
 
880
928
    def refs(self):
944
992
        LeafNode or InternalNode.
945
993
    """
946
994
 
 
995
    __slots__ = ('_node_width',)
 
996
 
947
997
    def __init__(self, prefix='', search_key_func=None):
948
998
        Node.__init__(self)
949
999
        # The size of an internalnode with default values and no children.
991
1041
        :param key: The key that the serialised node has.
992
1042
        :return: An InternalNode instance.
993
1043
        """
 
1044
        key = static_tuple.expect_static_tuple(key)
994
1045
        return _deserialise_internal_node(bytes, key,
995
1046
                                          search_key_func=search_key_func)
996
1047
 
1021
1072
            # for whatever we are missing
1022
1073
            shortcut = True
1023
1074
            for prefix, node in self._items.iteritems():
1024
 
                if node.__class__ is tuple:
 
1075
                if node.__class__ is StaticTuple:
1025
1076
                    keys[node] = (prefix, None)
1026
1077
                else:
1027
1078
                    yield node, None
1056
1107
                    # A given key can only match 1 child node, if it isn't
1057
1108
                    # there, then we can just return nothing
1058
1109
                    return
1059
 
                if node.__class__ is tuple:
 
1110
                if node.__class__ is StaticTuple:
1060
1111
                    keys[node] = (search_prefix, [key])
1061
1112
                else:
1062
1113
                    # This is loaded, and the only thing that can match,
1089
1140
                        # We can ignore this one
1090
1141
                        continue
1091
1142
                    node_key_filter = prefix_to_keys[search_prefix]
1092
 
                    if node.__class__ is tuple:
 
1143
                    if node.__class__ is StaticTuple:
1093
1144
                        keys[node] = (search_prefix, node_key_filter)
1094
1145
                    else:
1095
1146
                        yield node, node_key_filter
1104
1155
                        if sub_prefix in length_filter:
1105
1156
                            node_key_filter.extend(prefix_to_keys[sub_prefix])
1106
1157
                    if node_key_filter: # this key matched something, yield it
1107
 
                        if node.__class__ is tuple:
 
1158
                        if node.__class__ is StaticTuple:
1108
1159
                            keys[node] = (prefix, node_key_filter)
1109
1160
                        else:
1110
1161
                            yield node, node_key_filter
1113
1164
            found_keys = set()
1114
1165
            for key in keys:
1115
1166
                try:
1116
 
                    bytes = _page_cache[key]
 
1167
                    bytes = _get_cache()[key]
1117
1168
                except KeyError:
1118
1169
                    continue
1119
1170
                else:
1144
1195
                    prefix, node_key_filter = keys[record.key]
1145
1196
                    node_and_filters.append((node, node_key_filter))
1146
1197
                    self._items[prefix] = node
1147
 
                    _page_cache.add(record.key, bytes)
 
1198
                    _get_cache()[record.key] = bytes
1148
1199
                for info in node_and_filters:
1149
1200
                    yield info
1150
1201
 
1242
1293
        :return: An iterable of the keys inserted by this operation.
1243
1294
        """
1244
1295
        for node in self._items.itervalues():
1245
 
            if type(node) is tuple:
 
1296
            if type(node) is StaticTuple:
1246
1297
                # Never deserialised.
1247
1298
                continue
1248
1299
            if node._key is not None:
1259
1310
        lines.append('%s\n' % (self._search_prefix,))
1260
1311
        prefix_len = len(self._search_prefix)
1261
1312
        for prefix, node in sorted(self._items.items()):
1262
 
            if type(node) is tuple:
 
1313
            if type(node) is StaticTuple:
1263
1314
                key = node[0]
1264
1315
            else:
1265
1316
                key = node._key[0]
1269
1320
                    % (serialised, self._search_prefix))
1270
1321
            lines.append(serialised[prefix_len:])
1271
1322
        sha1, _, _ = store.add_lines((None,), (), lines)
1272
 
        self._key = ("sha1:" + sha1,)
1273
 
        _page_cache.add(self._key, ''.join(lines))
 
1323
        self._key = StaticTuple("sha1:" + sha1,).intern()
 
1324
        _get_cache()[self._key] = ''.join(lines)
1274
1325
        yield self._key
1275
1326
 
1276
1327
    def _search_key(self, key):
1304
1355
            raise AssertionError("unserialised nodes have no refs.")
1305
1356
        refs = []
1306
1357
        for value in self._items.itervalues():
1307
 
            if type(value) is tuple:
 
1358
            if type(value) is StaticTuple:
1308
1359
                refs.append(value)
1309
1360
            else:
1310
1361
                refs.append(value.key())
1320
1371
        return self._search_prefix
1321
1372
 
1322
1373
    def unmap(self, store, key, check_remap=True):
1323
 
        """Remove key from this node and it's children."""
 
1374
        """Remove key from this node and its children."""
1324
1375
        if not len(self._items):
1325
1376
            raise AssertionError("can't unmap in an empty InternalNode.")
1326
1377
        children = [node for node, _
1424
1475
 
1425
1476
    def __init__(self, store, new_root_keys, old_root_keys,
1426
1477
                 search_key_func, pb=None):
 
1478
        # TODO: Should we add a StaticTuple barrier here? It would be nice to
 
1479
        #       force callers to use StaticTuple, because there will often be
 
1480
        #       lots of keys passed in here. And even if we cast it locally,
 
1481
        #       that just meanst that we will have *both* a StaticTuple and a
 
1482
        #       tuple() in memory, referring to the same object. (so a net
 
1483
        #       increase in memory, not a decrease.)
1427
1484
        self._store = store
1428
1485
        self._new_root_keys = new_root_keys
1429
1486
        self._old_root_keys = old_root_keys
1431
1488
        # All uninteresting chks that we have seen. By the time they are added
1432
1489
        # here, they should be either fully ignored, or queued up for
1433
1490
        # processing
 
1491
        # TODO: This might grow to a large size if there are lots of merge
 
1492
        #       parents, etc. However, it probably doesn't scale to O(history)
 
1493
        #       like _processed_new_refs does.
1434
1494
        self._all_old_chks = set(self._old_root_keys)
1435
1495
        # All items that we have seen from the old_root_keys
1436
1496
        self._all_old_items = set()
1437
1497
        # These are interesting items which were either read, or already in the
1438
1498
        # interesting queue (so we don't need to walk them again)
 
1499
        # TODO: processed_new_refs becomes O(all_chks), consider switching to
 
1500
        #       SimpleSet here.
1439
1501
        self._processed_new_refs = set()
1440
1502
        self._search_key_func = search_key_func
1441
1503
 
1448
1510
        self._state = None
1449
1511
 
1450
1512
    def _read_nodes_from_store(self, keys):
1451
 
        # We chose not to use _page_cache, because we think in terms of records
1452
 
        # to be yielded. Also, we expect to touch each page only 1 time during
1453
 
        # this code. (We may want to evaluate saving the raw bytes into the
1454
 
        # page cache, which would allow a working tree update after the fetch
1455
 
        # to not have to read the bytes again.)
 
1513
        # We chose not to use _get_cache(), because we think in
 
1514
        # terms of records to be yielded. Also, we expect to touch each page
 
1515
        # only 1 time during this code. (We may want to evaluate saving the
 
1516
        # raw bytes into the page cache, which would allow a working tree
 
1517
        # update after the fetch to not have to read the bytes again.)
 
1518
        as_st = StaticTuple.from_sequence
1456
1519
        stream = self._store.get_record_stream(keys, 'unordered', True)
1457
1520
        for record in stream:
1458
1521
            if self._pb is not None:
1465
1528
            if type(node) is InternalNode:
1466
1529
                # Note we don't have to do node.refs() because we know that
1467
1530
                # there are no children that have been pushed into this node
 
1531
                # Note: Using as_st() here seemed to save 1.2MB, which would
 
1532
                #       indicate that we keep 100k prefix_refs around while
 
1533
                #       processing. They *should* be shorter lived than that...
 
1534
                #       It does cost us ~10s of processing time
 
1535
                #prefix_refs = [as_st(item) for item in node._items.iteritems()]
1468
1536
                prefix_refs = node._items.items()
1469
1537
                items = []
1470
1538
            else:
1471
1539
                prefix_refs = []
 
1540
                # Note: We don't use a StaticTuple here. Profiling showed a
 
1541
                #       minor memory improvement (0.8MB out of 335MB peak 0.2%)
 
1542
                #       But a significant slowdown (15s / 145s, or 10%)
1472
1543
                items = node._items.items()
1473
1544
            yield record, node, prefix_refs, items
1474
1545
 
1482
1553
                                if p_r[1] not in all_old_chks]
1483
1554
            new_refs = [p_r[1] for p_r in prefix_refs]
1484
1555
            all_old_chks.update(new_refs)
 
1556
            # TODO: This might be a good time to turn items into StaticTuple
 
1557
            #       instances and possibly intern them. However, this does not
 
1558
            #       impact 'initial branch' performance, so I'm not worrying
 
1559
            #       about this yet
1485
1560
            self._all_old_items.update(items)
1486
1561
            # Queue up the uninteresting references
1487
1562
            # Don't actually put them in the 'to-read' queue until we have
1540
1615
            #       current design allows for this, as callers will do the work
1541
1616
            #       to make the results unique. We might profile whether we
1542
1617
            #       gain anything by ensuring unique return values for items
 
1618
            # TODO: This might be a good time to cast to StaticTuple, as
 
1619
            #       self._new_item_queue will hold the contents of multiple
 
1620
            #       records for an extended lifetime
1543
1621
            new_items = [item for item in items
1544
1622
                               if item not in self._all_old_items]
1545
1623
            self._new_item_queue.extend(new_items)
1570
1648
        if new_items:
1571
1649
            yield None, new_items
1572
1650
        refs = refs.difference(all_old_chks)
 
1651
        processed_new_refs.update(refs)
1573
1652
        while refs:
 
1653
            # TODO: Using a SimpleSet for self._processed_new_refs and
 
1654
            #       saved as much as 10MB of peak memory. However, it requires
 
1655
            #       implementing a non-pyrex version.
1574
1656
            next_refs = set()
1575
1657
            next_refs_update = next_refs.update
1576
1658
            # Inlining _read_nodes_from_store improves 'bzr branch bzr.dev'
1577
1659
            # from 1m54s to 1m51s. Consider it.
1578
1660
            for record, _, p_refs, items in self._read_nodes_from_store(refs):
1579
 
                items = [item for item in items
1580
 
                         if item not in all_old_items]
 
1661
                if all_old_items:
 
1662
                    # using the 'if' check saves about 145s => 141s, when
 
1663
                    # streaming initial branch of Launchpad data.
 
1664
                    items = [item for item in items
 
1665
                             if item not in all_old_items]
1581
1666
                yield record, items
1582
1667
                next_refs_update([p_r[1] for p_r in p_refs])
 
1668
                del p_refs
 
1669
            # set1.difference(set/dict) walks all of set1, and checks if it
 
1670
            # exists in 'other'.
 
1671
            # set1.difference(iterable) walks all of iterable, and does a
 
1672
            # 'difference_update' on a clone of set1. Pick wisely based on the
 
1673
            # expected sizes of objects.
 
1674
            # in our case it is expected that 'new_refs' will always be quite
 
1675
            # small.
1583
1676
            next_refs = next_refs.difference(all_old_chks)
1584
1677
            next_refs = next_refs.difference(processed_new_refs)
1585
1678
            processed_new_refs.update(next_refs)
1592
1685
        self._old_queue = []
1593
1686
        all_old_chks = self._all_old_chks
1594
1687
        for record, _, prefix_refs, items in self._read_nodes_from_store(refs):
 
1688
            # TODO: Use StaticTuple here?
1595
1689
            self._all_old_items.update(items)
1596
1690
            refs = [r for _,r in prefix_refs if r not in all_old_chks]
1597
1691
            self._old_queue.extend(refs)
1632
1726
 
1633
1727
try:
1634
1728
    from bzrlib._chk_map_pyx import (
 
1729
        _bytes_to_text_key,
1635
1730
        _search_key_16,
1636
1731
        _search_key_255,
1637
1732
        _deserialise_leaf_node,
1638
1733
        _deserialise_internal_node,
1639
1734
        )
1640
 
except ImportError:
 
1735
except ImportError, e:
 
1736
    osutils.failed_to_load_extension(e)
1641
1737
    from bzrlib._chk_map_py import (
 
1738
        _bytes_to_text_key,
1642
1739
        _search_key_16,
1643
1740
        _search_key_255,
1644
1741
        _deserialise_leaf_node,
1646
1743
        )
1647
1744
search_key_registry.register('hash-16-way', _search_key_16)
1648
1745
search_key_registry.register('hash-255-way', _search_key_255)
 
1746
 
 
1747
 
 
1748
def _check_key(key):
 
1749
    """Helper function to assert that a key is properly formatted.
 
1750
 
 
1751
    This generally shouldn't be used in production code, but it can be helpful
 
1752
    to debug problems.
 
1753
    """
 
1754
    if type(key) is not StaticTuple:
 
1755
        raise TypeError('key %r is not StaticTuple but %s' % (key, type(key)))
 
1756
    if len(key) != 1:
 
1757
        raise ValueError('key %r should have length 1, not %d' % (key, len(key),))
 
1758
    if type(key[0]) is not str:
 
1759
        raise TypeError('key %r should hold a str, not %r'
 
1760
                        % (key, type(key[0])))
 
1761
    if not key[0].startswith('sha1:'):
 
1762
        raise ValueError('key %r should point to a sha1:' % (key,))
 
1763
 
 
1764