← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

(jam) Handle bug #382709 by encoding paths as 'mbcs' when spawning
        external diff.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/index.py
1
 
# Copyright (C) 2007-2011 Canonical Ltd
 
1
# Copyright (C) 2007, 2008 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
31
31
 
32
32
from bzrlib.lazy_import import lazy_import
33
33
lazy_import(globals(), """
34
 
from bzrlib import (
35
 
    bisect_multi,
36
 
    revision as _mod_revision,
37
 
    trace,
38
 
    )
 
34
from bzrlib import trace
 
35
from bzrlib.bisect_multi import bisect_multi_bytes
 
36
from bzrlib.revision import NULL_REVISION
 
37
from bzrlib.trace import mutter
39
38
""")
40
39
from bzrlib import (
41
40
    debug,
42
41
    errors,
43
42
    )
44
 
from bzrlib.static_tuple import StaticTuple
45
43
 
46
44
_HEADER_READV = (0, 200)
47
45
_OPTION_KEY_ELEMENTS = "key_elements="
70
68
class GraphIndexBuilder(object):
71
69
    """A builder that can build a GraphIndex.
72
70
 
73
 
    The resulting graph has the structure::
 
71
    The resulting graph has the structure:
74
72
 
75
 
      _SIGNATURE OPTIONS NODES NEWLINE
76
 
      _SIGNATURE     := 'Bazaar Graph Index 1' NEWLINE
77
 
      OPTIONS        := 'node_ref_lists=' DIGITS NEWLINE
78
 
      NODES          := NODE*
79
 
      NODE           := KEY NULL ABSENT? NULL REFERENCES NULL VALUE NEWLINE
80
 
      KEY            := Not-whitespace-utf8
81
 
      ABSENT         := 'a'
82
 
      REFERENCES     := REFERENCE_LIST (TAB REFERENCE_LIST){node_ref_lists - 1}
83
 
      REFERENCE_LIST := (REFERENCE (CR REFERENCE)*)?
84
 
      REFERENCE      := DIGITS  ; digits is the byte offset in the index of the
85
 
                                ; referenced key.
86
 
      VALUE          := no-newline-no-null-bytes
 
73
    _SIGNATURE OPTIONS NODES NEWLINE
 
74
    _SIGNATURE     := 'Bazaar Graph Index 1' NEWLINE
 
75
    OPTIONS        := 'node_ref_lists=' DIGITS NEWLINE
 
76
    NODES          := NODE*
 
77
    NODE           := KEY NULL ABSENT? NULL REFERENCES NULL VALUE NEWLINE
 
78
    KEY            := Not-whitespace-utf8
 
79
    ABSENT         := 'a'
 
80
    REFERENCES     := REFERENCE_LIST (TAB REFERENCE_LIST){node_ref_lists - 1}
 
81
    REFERENCE_LIST := (REFERENCE (CR REFERENCE)*)?
 
82
    REFERENCE      := DIGITS  ; digits is the byte offset in the index of the
 
83
                              ; referenced key.
 
84
    VALUE          := no-newline-no-null-bytes
87
85
    """
88
86
 
89
87
    def __init__(self, reference_lists=0, key_elements=1):
94
92
        :param key_elements: The number of bytestrings in each key.
95
93
        """
96
94
        self.reference_lists = reference_lists
 
95
        self._keys = set()
97
96
        # A dict of {key: (absent, ref_lists, value)}
98
97
        self._nodes = {}
99
 
        # Keys that are referenced but not actually present in this index
100
 
        self._absent_keys = set()
101
98
        self._nodes_by_key = None
102
99
        self._key_length = key_elements
103
100
        self._optimize_for_size = False
105
102
 
106
103
    def _check_key(self, key):
107
104
        """Raise BadIndexKey if key is not a valid key for this index."""
108
 
        if type(key) not in (tuple, StaticTuple):
 
105
        if type(key) != tuple:
109
106
            raise errors.BadIndexKey(key)
110
107
        if self._key_length != len(key):
111
108
            raise errors.BadIndexKey(key)
167
164
            return
168
165
        key_dict = self._nodes_by_key
169
166
        if self.reference_lists:
170
 
            key_value = StaticTuple(key, value, node_refs)
 
167
            key_value = key, value, node_refs
171
168
        else:
172
 
            key_value = StaticTuple(key, value)
 
169
            key_value = key, value
173
170
        for subkey in key[:-1]:
174
171
            key_dict = key_dict.setdefault(subkey, {})
175
172
        key_dict[key[-1]] = key_value
185
182
        :param value: The value associate with this key. Must not contain
186
183
            newlines or null characters.
187
184
        :return: (node_refs, absent_references)
188
 
        
189
 
            * node_refs: basically a packed form of 'references' where all
190
 
              iterables are tuples
191
 
            * absent_references: reference keys that are not in self._nodes.
192
 
              This may contain duplicates if the same key is referenced in
193
 
              multiple lists.
 
185
            node_refs   basically a packed form of 'references' where all
 
186
                        iterables are tuples
 
187
            absent_references   reference keys that are not in self._nodes.
 
188
                                This may contain duplicates if the same key is
 
189
                                referenced in multiple lists.
194
190
        """
195
 
        as_st = StaticTuple.from_sequence
196
191
        self._check_key(key)
197
192
        if _newline_null_re.search(value) is not None:
198
193
            raise errors.BadIndexValue(value)
207
202
                if reference not in self._nodes:
208
203
                    self._check_key(reference)
209
204
                    absent_references.append(reference)
210
 
            reference_list = as_st([as_st(ref).intern()
211
 
                                    for ref in reference_list])
212
 
            node_refs.append(reference_list)
213
 
        return as_st(node_refs), absent_references
 
205
            node_refs.append(tuple(reference_list))
 
206
        return tuple(node_refs), absent_references
214
207
 
215
208
    def add_node(self, key, value, references=()):
216
209
        """Add a node to the index.
221
214
        :param references: An iterable of iterables of keys. Each is a
222
215
            reference to another key.
223
216
        :param value: The value to associate with the key. It may be any
224
 
            bytes as long as it does not contain \\0 or \\n.
 
217
            bytes as long as it does not contain \0 or \n.
225
218
        """
226
219
        (node_refs,
227
220
         absent_references) = self._check_key_ref_value(key, references, value)
231
224
            # There may be duplicates, but I don't think it is worth worrying
232
225
            # about
233
226
            self._nodes[reference] = ('a', (), '')
234
 
        self._absent_keys.update(absent_references)
235
 
        self._absent_keys.discard(key)
236
227
        self._nodes[key] = ('', node_refs, value)
 
228
        self._keys.add(key)
237
229
        if self._nodes_by_key is not None and self._key_length > 1:
238
230
            self._update_nodes_by_key(key, value, node_refs)
239
231
 
240
 
    def clear_cache(self):
241
 
        """See GraphIndex.clear_cache()
242
 
 
243
 
        This is a no-op, but we need the api to conform to a generic 'Index'
244
 
        abstraction.
245
 
        """
246
 
        
247
232
    def finish(self):
248
 
        """Finish the index.
249
 
 
250
 
        :returns: cStringIO holding the full context of the index as it 
251
 
        should be written to disk.
252
 
        """
253
233
        lines = [_SIGNATURE]
254
234
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
255
235
        lines.append(_OPTION_KEY_ELEMENTS + str(self._key_length) + '\n')
256
 
        key_count = len(self._nodes) - len(self._absent_keys)
257
 
        lines.append(_OPTION_LEN + str(key_count) + '\n')
 
236
        lines.append(_OPTION_LEN + str(len(self._keys)) + '\n')
258
237
        prefix_length = sum(len(x) for x in lines)
259
238
        # references are byte offsets. To avoid having to do nasty
260
239
        # polynomial work to resolve offsets (references to later in the
354
333
        if combine_backing_indices is not None:
355
334
            self._combine_backing_indices = combine_backing_indices
356
335
 
357
 
    def find_ancestry(self, keys, ref_list_num):
358
 
        """See CombinedGraphIndex.find_ancestry()"""
359
 
        pending = set(keys)
360
 
        parent_map = {}
361
 
        missing_keys = set()
362
 
        while pending:
363
 
            next_pending = set()
364
 
            for _, key, value, ref_lists in self.iter_entries(pending):
365
 
                parent_keys = ref_lists[ref_list_num]
366
 
                parent_map[key] = parent_keys
367
 
                next_pending.update([p for p in parent_keys if p not in
368
 
                                     parent_map])
369
 
                missing_keys.update(pending.difference(parent_map))
370
 
            pending = next_pending
371
 
        return parent_map, missing_keys
372
 
 
373
336
 
374
337
class GraphIndex(object):
375
338
    """An index for data with embedded graphs.
389
352
    suitable for production use. :XXX
390
353
    """
391
354
 
392
 
    def __init__(self, transport, name, size, unlimited_cache=False, offset=0):
 
355
    def __init__(self, transport, name, size):
393
356
        """Open an index called name on transport.
394
357
 
395
358
        :param transport: A bzrlib.transport.Transport.
401
364
            avoided by having it supplied. If size is None, then bisection
402
365
            support will be disabled and accessing the index will just stream
403
366
            all the data.
404
 
        :param offset: Instead of starting the index data at offset 0, start it
405
 
            at an arbitrary offset.
406
367
        """
407
368
        self._transport = transport
408
369
        self._name = name
425
386
        self._size = size
426
387
        # The number of bytes we've read so far in trying to process this file
427
388
        self._bytes_read = 0
428
 
        self._base_offset = offset
429
389
 
430
390
    def __eq__(self, other):
431
391
        """Equal when self and other were created with the same parameters."""
451
411
            # We already did this
452
412
            return
453
413
        if 'index' in debug.debug_flags:
454
 
            trace.mutter('Reading entire index %s',
455
 
                          self._transport.abspath(self._name))
 
414
            mutter('Reading entire index %s', self._transport.abspath(self._name))
456
415
        if stream is None:
457
416
            stream = self._transport.get(self._name)
458
 
            if self._base_offset != 0:
459
 
                # This is wasteful, but it is better than dealing with
460
 
                # adjusting all the offsets, etc.
461
 
                stream = StringIO(stream.read()[self._base_offset:])
462
417
        self._read_prefix(stream)
463
418
        self._expected_elements = 3 + self._key_length
464
419
        line_count = 0
470
425
        trailers = 0
471
426
        pos = stream.tell()
472
427
        lines = stream.read().split('\n')
473
 
        # GZ 2009-09-20: Should really use a try/finally block to ensure close
474
 
        stream.close()
475
428
        del lines[-1]
476
429
        _, _, _, trailers = self._parse_lines(lines, pos)
477
430
        for key, absent, references, value in self._keys_by_offset.itervalues():
484
437
                node_value = value
485
438
            self._nodes[key] = node_value
486
439
        # cache the keys for quick set intersections
 
440
        self._keys = set(self._nodes)
487
441
        if trailers != 1:
488
442
            # there must be one line - the empty trailer line.
489
443
            raise errors.BadIndexData(self)
490
444
 
491
 
    def clear_cache(self):
492
 
        """Clear out any cached/memoized values.
493
 
 
494
 
        This can be called at any time, but generally it is used when we have
495
 
        extracted some information, but don't expect to be requesting any more
496
 
        from this index.
497
 
        """
498
 
 
499
445
    def external_references(self, ref_list_num):
500
446
        """Return references that are not present in this index.
501
447
        """
504
450
            raise ValueError('No ref list %d, index has %d ref lists'
505
451
                % (ref_list_num, self.node_ref_lists))
506
452
        refs = set()
507
 
        nodes = self._nodes
508
 
        for key, (value, ref_lists) in nodes.iteritems():
 
453
        for key, (value, ref_lists) in self._nodes.iteritems():
509
454
            ref_list = ref_lists[ref_list_num]
510
 
            refs.update([ref for ref in ref_list if ref not in nodes])
511
 
        return refs
 
455
            refs.update(ref_list)
 
456
        return refs - self._keys
512
457
 
513
458
    def _get_nodes_by_key(self):
514
459
        if self._nodes_by_key is None:
641
586
 
642
587
    def _iter_entries_from_total_buffer(self, keys):
643
588
        """Iterate over keys when the entire index is parsed."""
644
 
        # Note: See the note in BTreeBuilder.iter_entries for why we don't use
645
 
        #       .intersection() here
646
 
        nodes = self._nodes
647
 
        keys = [key for key in keys if key in nodes]
 
589
        keys = keys.intersection(self._keys)
648
590
        if self.node_ref_lists:
649
591
            for key in keys:
650
 
                value, node_refs = nodes[key]
 
592
                value, node_refs = self._nodes[key]
651
593
                yield self, key, value, node_refs
652
594
        else:
653
595
            for key in keys:
654
 
                yield self, key, nodes[key]
 
596
                yield self, key, self._nodes[key]
655
597
 
656
598
    def iter_entries(self, keys):
657
599
        """Iterate over keys within the index.
679
621
        if self._nodes is not None:
680
622
            return self._iter_entries_from_total_buffer(keys)
681
623
        else:
682
 
            return (result[1] for result in bisect_multi.bisect_multi_bytes(
 
624
            return (result[1] for result in bisect_multi_bytes(
683
625
                self._lookup_keys_via_location, self._size, keys))
684
626
 
685
627
    def iter_entries_prefix(self, keys):
760
702
                # the last thing looked up was a terminal element
761
703
                yield (self, ) + key_dict
762
704
 
763
 
    def _find_ancestors(self, keys, ref_list_num, parent_map, missing_keys):
764
 
        """See BTreeIndex._find_ancestors."""
765
 
        # The api can be implemented as a trivial overlay on top of
766
 
        # iter_entries, it is not an efficient implementation, but it at least
767
 
        # gets the job done.
768
 
        found_keys = set()
769
 
        search_keys = set()
770
 
        for index, key, value, refs in self.iter_entries(keys):
771
 
            parent_keys = refs[ref_list_num]
772
 
            found_keys.add(key)
773
 
            parent_map[key] = parent_keys
774
 
            search_keys.update(parent_keys)
775
 
        # Figure out what, if anything, was missing
776
 
        missing_keys.update(set(keys).difference(found_keys))
777
 
        search_keys = search_keys.difference(parent_map)
778
 
        return search_keys
779
 
 
780
705
    def key_count(self):
781
706
        """Return an estimate of the number of keys in this index.
782
707
 
1194
1119
            self._parsed_key_map.insert(index + 1, new_key)
1195
1120
 
1196
1121
    def _read_and_parse(self, readv_ranges):
1197
 
        """Read the ranges and parse the resulting data.
 
1122
        """Read the the ranges and parse the resulting data.
1198
1123
 
1199
1124
        :param readv_ranges: A prepared readv range list.
1200
1125
        """
1206
1131
            self._buffer_all()
1207
1132
            return
1208
1133
 
1209
 
        base_offset = self._base_offset
1210
 
        if base_offset != 0:
1211
 
            # Rewrite the ranges for the offset
1212
 
            readv_ranges = [(start+base_offset, size)
1213
 
                            for start, size in readv_ranges]
1214
1134
        readv_data = self._transport.readv(self._name, readv_ranges, True,
1215
 
            self._size + self._base_offset)
 
1135
            self._size)
1216
1136
        # parse
1217
1137
        for offset, data in readv_data:
1218
 
            offset -= base_offset
1219
1138
            self._bytes_read += len(data)
1220
 
            if offset < 0:
1221
 
                # transport.readv() expanded to extra data which isn't part of
1222
 
                # this index
1223
 
                data = data[-offset:]
1224
 
                offset = 0
1225
1139
            if offset == 0 and len(data) == self._size:
1226
1140
                # We read the whole range, most likely because the
1227
1141
                # Transport upcast our readv ranges into one long request
1254
1168
    static data.
1255
1169
 
1256
1170
    Queries against the combined index will be made against the first index,
1257
 
    and then the second and so on. The order of indices can thus influence
 
1171
    and then the second and so on. The order of index's can thus influence
1258
1172
    performance significantly. For example, if one index is on local disk and a
1259
1173
    second on a remote server, the local disk index should be before the other
1260
1174
    in the index list.
1261
 
    
1262
 
    Also, queries tend to need results from the same indices as previous
1263
 
    queries.  So the indices will be reordered after every query to put the
1264
 
    indices that had the result(s) of that query first (while otherwise
1265
 
    preserving the relative ordering).
1266
1175
    """
1267
1176
 
1268
1177
    def __init__(self, indices, reload_func=None):
1275
1184
        """
1276
1185
        self._indices = indices
1277
1186
        self._reload_func = reload_func
1278
 
        # Sibling indices are other CombinedGraphIndex that we should call
1279
 
        # _move_to_front_by_name on when we auto-reorder ourself.
1280
 
        self._sibling_indices = []
1281
 
        # A list of names that corresponds to the instances in self._indices,
1282
 
        # so _index_names[0] is always the name for _indices[0], etc.  Sibling
1283
 
        # indices must all use the same set of names as each other.
1284
 
        self._index_names = [None] * len(self._indices)
1285
1187
 
1286
1188
    def __repr__(self):
1287
1189
        return "%s(%s)" % (
1288
1190
                self.__class__.__name__,
1289
1191
                ', '.join(map(repr, self._indices)))
1290
1192
 
1291
 
    def clear_cache(self):
1292
 
        """See GraphIndex.clear_cache()"""
1293
 
        for index in self._indices:
1294
 
            index.clear_cache()
1295
 
 
1296
1193
    def get_parent_map(self, keys):
1297
1194
        """See graph.StackedParentsProvider.get_parent_map"""
1298
1195
        search_keys = set(keys)
1299
 
        if _mod_revision.NULL_REVISION in search_keys:
1300
 
            search_keys.discard(_mod_revision.NULL_REVISION)
1301
 
            found_parents = {_mod_revision.NULL_REVISION:[]}
 
1196
        if NULL_REVISION in search_keys:
 
1197
            search_keys.discard(NULL_REVISION)
 
1198
            found_parents = {NULL_REVISION:[]}
1302
1199
        else:
1303
1200
            found_parents = {}
1304
1201
        for index, key, value, refs in self.iter_entries(search_keys):
1305
1202
            parents = refs[0]
1306
1203
            if not parents:
1307
 
                parents = (_mod_revision.NULL_REVISION,)
 
1204
                parents = (NULL_REVISION,)
1308
1205
            found_parents[key] = parents
1309
1206
        return found_parents
1310
1207
 
1311
1208
    has_key = _has_key_from_parent_map
1312
1209
 
1313
 
    def insert_index(self, pos, index, name=None):
 
1210
    def insert_index(self, pos, index):
1314
1211
        """Insert a new index in the list of indices to query.
1315
1212
 
1316
1213
        :param pos: The position to insert the index.
1317
1214
        :param index: The index to insert.
1318
 
        :param name: a name for this index, e.g. a pack name.  These names can
1319
 
            be used to reflect index reorderings to related CombinedGraphIndex
1320
 
            instances that use the same names.  (see set_sibling_indices)
1321
1215
        """
1322
1216
        self._indices.insert(pos, index)
1323
 
        self._index_names.insert(pos, name)
1324
1217
 
1325
1218
    def iter_all_entries(self):
1326
1219
        """Iterate over all keys within the index
1351
1244
        value and are only reported once.
1352
1245
 
1353
1246
        :param keys: An iterable providing the keys to be retrieved.
1354
 
        :return: An iterable of (index, key, reference_lists, value). There is
1355
 
            no defined order for the result iteration - it will be in the most
 
1247
        :return: An iterable of (index, key, reference_lists, value). There is no
 
1248
            defined order for the result iteration - it will be in the most
1356
1249
            efficient order for the index.
1357
1250
        """
1358
1251
        keys = set(keys)
1359
 
        hit_indices = []
1360
1252
        while True:
1361
1253
            try:
1362
1254
                for index in self._indices:
1363
1255
                    if not keys:
1364
 
                        break
1365
 
                    index_hit = False
 
1256
                        return
1366
1257
                    for node in index.iter_entries(keys):
1367
1258
                        keys.remove(node[1])
1368
1259
                        yield node
1369
 
                        index_hit = True
1370
 
                    if index_hit:
1371
 
                        hit_indices.append(index)
1372
 
                break
 
1260
                return
1373
1261
            except errors.NoSuchFile:
1374
1262
                self._reload_or_raise()
1375
 
        self._move_to_front(hit_indices)
1376
1263
 
1377
1264
    def iter_entries_prefix(self, keys):
1378
1265
        """Iterate over keys within the index using prefix matching.
1398
1285
        if not keys:
1399
1286
            return
1400
1287
        seen_keys = set()
1401
 
        hit_indices = []
1402
1288
        while True:
1403
1289
            try:
1404
1290
                for index in self._indices:
1405
 
                    index_hit = False
1406
1291
                    for node in index.iter_entries_prefix(keys):
1407
1292
                        if node[1] in seen_keys:
1408
1293
                            continue
1409
1294
                        seen_keys.add(node[1])
1410
1295
                        yield node
1411
 
                        index_hit = True
1412
 
                    if index_hit:
1413
 
                        hit_indices.append(index)
1414
 
                break
 
1296
                return
1415
1297
            except errors.NoSuchFile:
1416
1298
                self._reload_or_raise()
1417
 
        self._move_to_front(hit_indices)
1418
 
 
1419
 
    def _move_to_front(self, hit_indices):
1420
 
        """Rearrange self._indices so that hit_indices are first.
1421
 
 
1422
 
        Order is maintained as much as possible, e.g. the first unhit index
1423
 
        will be the first index in _indices after the hit_indices, and the
1424
 
        hit_indices will be present in exactly the order they are passed to
1425
 
        _move_to_front.
1426
 
 
1427
 
        _move_to_front propagates to all objects in self._sibling_indices by
1428
 
        calling _move_to_front_by_name.
1429
 
        """
1430
 
        if self._indices[:len(hit_indices)] == hit_indices:
1431
 
            # The 'hit_indices' are already at the front (and in the same
1432
 
            # order), no need to re-order
1433
 
            return
1434
 
        hit_names = self._move_to_front_by_index(hit_indices)
1435
 
        for sibling_idx in self._sibling_indices:
1436
 
            sibling_idx._move_to_front_by_name(hit_names)
1437
 
 
1438
 
    def _move_to_front_by_index(self, hit_indices):
1439
 
        """Core logic for _move_to_front.
1440
 
        
1441
 
        Returns a list of names corresponding to the hit_indices param.
1442
 
        """
1443
 
        indices_info = zip(self._index_names, self._indices)
1444
 
        if 'index' in debug.debug_flags:
1445
 
            trace.mutter('CombinedGraphIndex reordering: currently %r, '
1446
 
                         'promoting %r', indices_info, hit_indices)
1447
 
        hit_names = []
1448
 
        unhit_names = []
1449
 
        new_hit_indices = []
1450
 
        unhit_indices = []
1451
 
 
1452
 
        for offset, (name, idx) in enumerate(indices_info):
1453
 
            if idx in hit_indices:
1454
 
                hit_names.append(name)
1455
 
                new_hit_indices.append(idx)
1456
 
                if len(new_hit_indices) == len(hit_indices):
1457
 
                    # We've found all of the hit entries, everything else is
1458
 
                    # unhit
1459
 
                    unhit_names.extend(self._index_names[offset+1:])
1460
 
                    unhit_indices.extend(self._indices[offset+1:])
1461
 
                    break
1462
 
            else:
1463
 
                unhit_names.append(name)
1464
 
                unhit_indices.append(idx)
1465
 
 
1466
 
        self._indices = new_hit_indices + unhit_indices
1467
 
        self._index_names = hit_names + unhit_names
1468
 
        if 'index' in debug.debug_flags:
1469
 
            trace.mutter('CombinedGraphIndex reordered: %r', self._indices)
1470
 
        return hit_names
1471
 
 
1472
 
    def _move_to_front_by_name(self, hit_names):
1473
 
        """Moves indices named by 'hit_names' to front of the search order, as
1474
 
        described in _move_to_front.
1475
 
        """
1476
 
        # Translate names to index instances, and then call
1477
 
        # _move_to_front_by_index.
1478
 
        indices_info = zip(self._index_names, self._indices)
1479
 
        hit_indices = []
1480
 
        for name, idx in indices_info:
1481
 
            if name in hit_names:
1482
 
                hit_indices.append(idx)
1483
 
        self._move_to_front_by_index(hit_indices)
1484
 
 
1485
 
    def find_ancestry(self, keys, ref_list_num):
1486
 
        """Find the complete ancestry for the given set of keys.
1487
 
 
1488
 
        Note that this is a whole-ancestry request, so it should be used
1489
 
        sparingly.
1490
 
 
1491
 
        :param keys: An iterable of keys to look for
1492
 
        :param ref_list_num: The reference list which references the parents
1493
 
            we care about.
1494
 
        :return: (parent_map, missing_keys)
1495
 
        """
1496
 
        # XXX: make this call _move_to_front?
1497
 
        missing_keys = set()
1498
 
        parent_map = {}
1499
 
        keys_to_lookup = set(keys)
1500
 
        generation = 0
1501
 
        while keys_to_lookup:
1502
 
            # keys that *all* indexes claim are missing, stop searching them
1503
 
            generation += 1
1504
 
            all_index_missing = None
1505
 
            # print 'gen\tidx\tsub\tn_keys\tn_pmap\tn_miss'
1506
 
            # print '%4d\t\t\t%4d\t%5d\t%5d' % (generation, len(keys_to_lookup),
1507
 
            #                                   len(parent_map),
1508
 
            #                                   len(missing_keys))
1509
 
            for index_idx, index in enumerate(self._indices):
1510
 
                # TODO: we should probably be doing something with
1511
 
                #       'missing_keys' since we've already determined that
1512
 
                #       those revisions have not been found anywhere
1513
 
                index_missing_keys = set()
1514
 
                # Find all of the ancestry we can from this index
1515
 
                # keep looking until the search_keys set is empty, which means
1516
 
                # things we didn't find should be in index_missing_keys
1517
 
                search_keys = keys_to_lookup
1518
 
                sub_generation = 0
1519
 
                # print '    \t%2d\t\t%4d\t%5d\t%5d' % (
1520
 
                #     index_idx, len(search_keys),
1521
 
                #     len(parent_map), len(index_missing_keys))
1522
 
                while search_keys:
1523
 
                    sub_generation += 1
1524
 
                    # TODO: ref_list_num should really be a parameter, since
1525
 
                    #       CombinedGraphIndex does not know what the ref lists
1526
 
                    #       mean.
1527
 
                    search_keys = index._find_ancestors(search_keys,
1528
 
                        ref_list_num, parent_map, index_missing_keys)
1529
 
                    # print '    \t  \t%2d\t%4d\t%5d\t%5d' % (
1530
 
                    #     sub_generation, len(search_keys),
1531
 
                    #     len(parent_map), len(index_missing_keys))
1532
 
                # Now set whatever was missing to be searched in the next index
1533
 
                keys_to_lookup = index_missing_keys
1534
 
                if all_index_missing is None:
1535
 
                    all_index_missing = set(index_missing_keys)
1536
 
                else:
1537
 
                    all_index_missing.intersection_update(index_missing_keys)
1538
 
                if not keys_to_lookup:
1539
 
                    break
1540
 
            if all_index_missing is None:
1541
 
                # There were no indexes, so all search keys are 'missing'
1542
 
                missing_keys.update(keys_to_lookup)
1543
 
                keys_to_lookup = None
1544
 
            else:
1545
 
                missing_keys.update(all_index_missing)
1546
 
                keys_to_lookup.difference_update(all_index_missing)
1547
 
        return parent_map, missing_keys
1548
1299
 
1549
1300
    def key_count(self):
1550
1301
        """Return an estimate of the number of keys in this index.
1579
1330
                         ' Raising original exception.')
1580
1331
            raise exc_type, exc_value, exc_traceback
1581
1332
 
1582
 
    def set_sibling_indices(self, sibling_combined_graph_indices):
1583
 
        """Set the CombinedGraphIndex objects to reorder after reordering self.
1584
 
        """
1585
 
        self._sibling_indices = sibling_combined_graph_indices
1586
 
 
1587
1333
    def validate(self):
1588
1334
        """Validate that everything in the index can be accessed."""
1589
1335
        while True:
1642
1388
            defined order for the result iteration - it will be in the most
1643
1389
            efficient order for the index (keys iteration order in this case).
1644
1390
        """
1645
 
        # Note: See BTreeBuilder.iter_entries for an explanation of why we
1646
 
        #       aren't using set().intersection() here
1647
 
        nodes = self._nodes
1648
 
        keys = [key for key in keys if key in nodes]
 
1391
        keys = set(keys)
1649
1392
        if self.reference_lists:
1650
 
            for key in keys:
1651
 
                node = nodes[key]
 
1393
            for key in keys.intersection(self._keys):
 
1394
                node = self._nodes[key]
1652
1395
                if not node[0]:
1653
1396
                    yield self, key, node[2], node[1]
1654
1397
        else:
1655
 
            for key in keys:
1656
 
                node = nodes[key]
 
1398
            for key in keys.intersection(self._keys):
 
1399
                node = self._nodes[key]
1657
1400
                if not node[0]:
1658
1401
                    yield self, key, node[2]
1659
1402
 
1733
1476
 
1734
1477
        For InMemoryGraphIndex the estimate is exact.
1735
1478
        """
1736
 
        return len(self._nodes) - len(self._absent_keys)
 
1479
        return len(self._keys)
1737
1480
 
1738
1481
    def validate(self):
1739
1482
        """In memory index's have no known corruption at the moment."""