/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/index.py
1
 
# Copyright (C) 2007, 2008 Canonical Ltd
 
1
# Copyright (C) 2007-2010 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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
"""Indexing facilities."""
18
18
 
39
39
from bzrlib import (
40
40
    debug,
41
41
    errors,
42
 
    symbol_versioning,
43
42
    )
 
43
from bzrlib.static_tuple import StaticTuple
44
44
 
45
45
_HEADER_READV = (0, 200)
46
46
_OPTION_KEY_ELEMENTS = "key_elements="
93
93
        :param key_elements: The number of bytestrings in each key.
94
94
        """
95
95
        self.reference_lists = reference_lists
96
 
        self._keys = set()
97
96
        # A dict of {key: (absent, ref_lists, value)}
98
97
        self._nodes = {}
 
98
        # Keys that are referenced but not actually present in this index
 
99
        self._absent_keys = set()
99
100
        self._nodes_by_key = None
100
101
        self._key_length = key_elements
101
102
        self._optimize_for_size = False
 
103
        self._combine_backing_indices = True
102
104
 
103
105
    def _check_key(self, key):
104
106
        """Raise BadIndexKey if key is not a valid key for this index."""
105
 
        if type(key) != tuple:
 
107
        if type(key) not in (tuple, StaticTuple):
106
108
            raise errors.BadIndexKey(key)
107
109
        if self._key_length != len(key):
108
110
            raise errors.BadIndexKey(key)
164
166
            return
165
167
        key_dict = self._nodes_by_key
166
168
        if self.reference_lists:
167
 
            key_value = key, value, node_refs
 
169
            key_value = StaticTuple(key, value, node_refs)
168
170
        else:
169
 
            key_value = key, value
 
171
            key_value = StaticTuple(key, value)
170
172
        for subkey in key[:-1]:
171
173
            key_dict = key_dict.setdefault(subkey, {})
172
174
        key_dict[key[-1]] = key_value
188
190
                                This may contain duplicates if the same key is
189
191
                                referenced in multiple lists.
190
192
        """
 
193
        as_st = StaticTuple.from_sequence
191
194
        self._check_key(key)
192
195
        if _newline_null_re.search(value) is not None:
193
196
            raise errors.BadIndexValue(value)
202
205
                if reference not in self._nodes:
203
206
                    self._check_key(reference)
204
207
                    absent_references.append(reference)
205
 
            node_refs.append(tuple(reference_list))
206
 
        return tuple(node_refs), absent_references
 
208
            reference_list = as_st([as_st(ref).intern()
 
209
                                    for ref in reference_list])
 
210
            node_refs.append(reference_list)
 
211
        return as_st(node_refs), absent_references
207
212
 
208
213
    def add_node(self, key, value, references=()):
209
214
        """Add a node to the index.
224
229
            # There may be duplicates, but I don't think it is worth worrying
225
230
            # about
226
231
            self._nodes[reference] = ('a', (), '')
 
232
        self._absent_keys.update(absent_references)
 
233
        self._absent_keys.discard(key)
227
234
        self._nodes[key] = ('', node_refs, value)
228
 
        self._keys.add(key)
229
235
        if self._nodes_by_key is not None and self._key_length > 1:
230
236
            self._update_nodes_by_key(key, value, node_refs)
231
237
 
 
238
    def clear_cache(self):
 
239
        """See GraphIndex.clear_cache()
 
240
 
 
241
        This is a no-op, but we need the api to conform to a generic 'Index'
 
242
        abstraction.
 
243
        """
 
244
        
232
245
    def finish(self):
233
246
        lines = [_SIGNATURE]
234
247
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
235
248
        lines.append(_OPTION_KEY_ELEMENTS + str(self._key_length) + '\n')
236
 
        lines.append(_OPTION_LEN + str(len(self._keys)) + '\n')
 
249
        key_count = len(self._nodes) - len(self._absent_keys)
 
250
        lines.append(_OPTION_LEN + str(key_count) + '\n')
237
251
        prefix_length = sum(len(x) for x in lines)
238
252
        # references are byte offsets. To avoid having to do nasty
239
253
        # polynomial work to resolve offsets (references to later in the
315
329
                (len(result.getvalue()), expected_bytes))
316
330
        return result
317
331
 
318
 
    def set_optimize(self, for_size=True):
 
332
    def set_optimize(self, for_size=None, combine_backing_indices=None):
319
333
        """Change how the builder tries to optimize the result.
320
334
 
321
335
        :param for_size: Tell the builder to try and make the index as small as
322
336
            possible.
 
337
        :param combine_backing_indices: If the builder spills to disk to save
 
338
            memory, should the on-disk indices be combined. Set to True if you
 
339
            are going to be probing the index, but to False if you are not. (If
 
340
            you are not querying, then the time spent combining is wasted.)
323
341
        :return: None
324
342
        """
325
343
        # GraphIndexBuilder itself doesn't pay attention to the flag yet, but
326
344
        # other builders do.
327
 
        self._optimize_for_size = for_size
 
345
        if for_size is not None:
 
346
            self._optimize_for_size = for_size
 
347
        if combine_backing_indices is not None:
 
348
            self._combine_backing_indices = combine_backing_indices
 
349
 
 
350
    def find_ancestry(self, keys, ref_list_num):
 
351
        """See CombinedGraphIndex.find_ancestry()"""
 
352
        pending = set(keys)
 
353
        parent_map = {}
 
354
        missing_keys = set()
 
355
        while pending:
 
356
            next_pending = set()
 
357
            for _, key, value, ref_lists in self.iter_entries(pending):
 
358
                parent_keys = ref_lists[ref_list_num]
 
359
                parent_map[key] = parent_keys
 
360
                next_pending.update([p for p in parent_keys if p not in
 
361
                                     parent_map])
 
362
                missing_keys.update(pending.difference(parent_map))
 
363
            pending = next_pending
 
364
        return parent_map, missing_keys
328
365
 
329
366
 
330
367
class GraphIndex(object):
345
382
    suitable for production use. :XXX
346
383
    """
347
384
 
348
 
    def __init__(self, transport, name, size):
 
385
    def __init__(self, transport, name, size, unlimited_cache=False, offset=0):
349
386
        """Open an index called name on transport.
350
387
 
351
388
        :param transport: A bzrlib.transport.Transport.
357
394
            avoided by having it supplied. If size is None, then bisection
358
395
            support will be disabled and accessing the index will just stream
359
396
            all the data.
 
397
        :param offset: Instead of starting the index data at offset 0, start it
 
398
            at an arbitrary offset.
360
399
        """
361
400
        self._transport = transport
362
401
        self._name = name
379
418
        self._size = size
380
419
        # The number of bytes we've read so far in trying to process this file
381
420
        self._bytes_read = 0
 
421
        self._base_offset = offset
382
422
 
383
423
    def __eq__(self, other):
384
424
        """Equal when self and other were created with the same parameters."""
407
447
            mutter('Reading entire index %s', self._transport.abspath(self._name))
408
448
        if stream is None:
409
449
            stream = self._transport.get(self._name)
 
450
            if self._base_offset != 0:
 
451
                # This is wasteful, but it is better than dealing with
 
452
                # adjusting all the offsets, etc.
 
453
                stream = StringIO(stream.read()[self._base_offset:])
410
454
        self._read_prefix(stream)
411
455
        self._expected_elements = 3 + self._key_length
412
456
        line_count = 0
418
462
        trailers = 0
419
463
        pos = stream.tell()
420
464
        lines = stream.read().split('\n')
 
465
        stream.close()
421
466
        del lines[-1]
422
467
        _, _, _, trailers = self._parse_lines(lines, pos)
423
468
        for key, absent, references, value in self._keys_by_offset.itervalues():
430
475
                node_value = value
431
476
            self._nodes[key] = node_value
432
477
        # cache the keys for quick set intersections
433
 
        self._keys = set(self._nodes)
434
478
        if trailers != 1:
435
479
            # there must be one line - the empty trailer line.
436
480
            raise errors.BadIndexData(self)
437
481
 
 
482
    def clear_cache(self):
 
483
        """Clear out any cached/memoized values.
 
484
 
 
485
        This can be called at any time, but generally it is used when we have
 
486
        extracted some information, but don't expect to be requesting any more
 
487
        from this index.
 
488
        """
 
489
 
438
490
    def external_references(self, ref_list_num):
439
491
        """Return references that are not present in this index.
440
492
        """
443
495
            raise ValueError('No ref list %d, index has %d ref lists'
444
496
                % (ref_list_num, self.node_ref_lists))
445
497
        refs = set()
446
 
        for key, (value, ref_lists) in self._nodes.iteritems():
 
498
        nodes = self._nodes
 
499
        for key, (value, ref_lists) in nodes.iteritems():
447
500
            ref_list = ref_lists[ref_list_num]
448
 
            refs.update(ref_list)
449
 
        return refs - self._keys
 
501
            refs.update([ref for ref in ref_list if ref not in nodes])
 
502
        return refs
450
503
 
451
504
    def _get_nodes_by_key(self):
452
505
        if self._nodes_by_key is None:
579
632
 
580
633
    def _iter_entries_from_total_buffer(self, keys):
581
634
        """Iterate over keys when the entire index is parsed."""
582
 
        keys = keys.intersection(self._keys)
 
635
        # Note: See the note in BTreeBuilder.iter_entries for why we don't use
 
636
        #       .intersection() here
 
637
        nodes = self._nodes
 
638
        keys = [key for key in keys if key in nodes]
583
639
        if self.node_ref_lists:
584
640
            for key in keys:
585
 
                value, node_refs = self._nodes[key]
 
641
                value, node_refs = nodes[key]
586
642
                yield self, key, value, node_refs
587
643
        else:
588
644
            for key in keys:
589
 
                yield self, key, self._nodes[key]
 
645
                yield self, key, nodes[key]
590
646
 
591
647
    def iter_entries(self, keys):
592
648
        """Iterate over keys within the index.
695
751
                # the last thing looked up was a terminal element
696
752
                yield (self, ) + key_dict
697
753
 
 
754
    def _find_ancestors(self, keys, ref_list_num, parent_map, missing_keys):
 
755
        """See BTreeIndex._find_ancestors."""
 
756
        # The api can be implemented as a trivial overlay on top of
 
757
        # iter_entries, it is not an efficient implementation, but it at least
 
758
        # gets the job done.
 
759
        found_keys = set()
 
760
        search_keys = set()
 
761
        for index, key, value, refs in self.iter_entries(keys):
 
762
            parent_keys = refs[ref_list_num]
 
763
            found_keys.add(key)
 
764
            parent_map[key] = parent_keys
 
765
            search_keys.update(parent_keys)
 
766
        # Figure out what, if anything, was missing
 
767
        missing_keys.update(set(keys).difference(found_keys))
 
768
        search_keys = search_keys.difference(parent_map)
 
769
        return search_keys
 
770
 
698
771
    def key_count(self):
699
772
        """Return an estimate of the number of keys in this index.
700
773
 
1112
1185
            self._parsed_key_map.insert(index + 1, new_key)
1113
1186
 
1114
1187
    def _read_and_parse(self, readv_ranges):
1115
 
        """Read the the ranges and parse the resulting data.
 
1188
        """Read the ranges and parse the resulting data.
1116
1189
 
1117
1190
        :param readv_ranges: A prepared readv range list.
1118
1191
        """
1124
1197
            self._buffer_all()
1125
1198
            return
1126
1199
 
 
1200
        base_offset = self._base_offset
 
1201
        if base_offset != 0:
 
1202
            # Rewrite the ranges for the offset
 
1203
            readv_ranges = [(start+base_offset, size)
 
1204
                            for start, size in readv_ranges]
1127
1205
        readv_data = self._transport.readv(self._name, readv_ranges, True,
1128
 
            self._size)
 
1206
            self._size + self._base_offset)
1129
1207
        # parse
1130
1208
        for offset, data in readv_data:
 
1209
            offset -= base_offset
1131
1210
            self._bytes_read += len(data)
 
1211
            if offset < 0:
 
1212
                # transport.readv() expanded to extra data which isn't part of
 
1213
                # this index
 
1214
                data = data[-offset:]
 
1215
                offset = 0
1132
1216
            if offset == 0 and len(data) == self._size:
1133
1217
                # We read the whole range, most likely because the
1134
1218
                # Transport upcast our readv ranges into one long request
1161
1245
    static data.
1162
1246
 
1163
1247
    Queries against the combined index will be made against the first index,
1164
 
    and then the second and so on. The order of index's can thus influence
 
1248
    and then the second and so on. The order of indices can thus influence
1165
1249
    performance significantly. For example, if one index is on local disk and a
1166
1250
    second on a remote server, the local disk index should be before the other
1167
1251
    in the index list.
 
1252
    
 
1253
    Also, queries tend to need results from the same indices as previous
 
1254
    queries.  So the indices will be reordered after every query to put the
 
1255
    indices that had the result(s) of that query first (while otherwise
 
1256
    preserving the relative ordering).
1168
1257
    """
1169
1258
 
1170
1259
    def __init__(self, indices, reload_func=None):
1177
1266
        """
1178
1267
        self._indices = indices
1179
1268
        self._reload_func = reload_func
 
1269
        # Sibling indices are other CombinedGraphIndex that we should call
 
1270
        # _move_to_front_by_name on when we auto-reorder ourself.
 
1271
        self._sibling_indices = []
 
1272
        # A list of names that corresponds to the instances in self._indices,
 
1273
        # so _index_names[0] is always the name for _indices[0], etc.  Sibling
 
1274
        # indices must all use the same set of names as each other.
 
1275
        self._index_names = [None] * len(self._indices)
1180
1276
 
1181
1277
    def __repr__(self):
1182
1278
        return "%s(%s)" % (
1183
1279
                self.__class__.__name__,
1184
1280
                ', '.join(map(repr, self._indices)))
1185
1281
 
1186
 
    @symbol_versioning.deprecated_method(symbol_versioning.one_one)
1187
 
    def get_parents(self, revision_ids):
1188
 
        """See graph._StackedParentsProvider.get_parents.
1189
 
 
1190
 
        This implementation thunks the graph.Graph.get_parents api across to
1191
 
        GraphIndex.
1192
 
 
1193
 
        :param revision_ids: An iterable of graph keys for this graph.
1194
 
        :return: A list of parent details for each key in revision_ids.
1195
 
            Each parent details will be one of:
1196
 
             * None when the key was missing
1197
 
             * (NULL_REVISION,) when the key has no parents.
1198
 
             * (parent_key, parent_key...) otherwise.
1199
 
        """
1200
 
        parent_map = self.get_parent_map(revision_ids)
1201
 
        return [parent_map.get(r, None) for r in revision_ids]
 
1282
    def clear_cache(self):
 
1283
        """See GraphIndex.clear_cache()"""
 
1284
        for index in self._indices:
 
1285
            index.clear_cache()
1202
1286
 
1203
1287
    def get_parent_map(self, keys):
1204
 
        """See graph._StackedParentsProvider.get_parent_map"""
 
1288
        """See graph.StackedParentsProvider.get_parent_map"""
1205
1289
        search_keys = set(keys)
1206
1290
        if NULL_REVISION in search_keys:
1207
1291
            search_keys.discard(NULL_REVISION)
1217
1301
 
1218
1302
    has_key = _has_key_from_parent_map
1219
1303
 
1220
 
    def insert_index(self, pos, index):
 
1304
    def insert_index(self, pos, index, name=None):
1221
1305
        """Insert a new index in the list of indices to query.
1222
1306
 
1223
1307
        :param pos: The position to insert the index.
1224
1308
        :param index: The index to insert.
 
1309
        :param name: a name for this index, e.g. a pack name.  These names can
 
1310
            be used to reflect index reorderings to related CombinedGraphIndex
 
1311
            instances that use the same names.  (see set_sibling_indices)
1225
1312
        """
1226
1313
        self._indices.insert(pos, index)
 
1314
        self._index_names.insert(pos, name)
1227
1315
 
1228
1316
    def iter_all_entries(self):
1229
1317
        """Iterate over all keys within the index
1254
1342
        value and are only reported once.
1255
1343
 
1256
1344
        :param keys: An iterable providing the keys to be retrieved.
1257
 
        :return: An iterable of (index, key, reference_lists, value). There is no
1258
 
            defined order for the result iteration - it will be in the most
 
1345
        :return: An iterable of (index, key, reference_lists, value). There is
 
1346
            no defined order for the result iteration - it will be in the most
1259
1347
            efficient order for the index.
1260
1348
        """
1261
1349
        keys = set(keys)
 
1350
        hit_indices = []
1262
1351
        while True:
1263
1352
            try:
1264
1353
                for index in self._indices:
1265
1354
                    if not keys:
1266
 
                        return
 
1355
                        break
 
1356
                    index_hit = False
1267
1357
                    for node in index.iter_entries(keys):
1268
1358
                        keys.remove(node[1])
1269
1359
                        yield node
1270
 
                return
 
1360
                        index_hit = True
 
1361
                    if index_hit:
 
1362
                        hit_indices.append(index)
 
1363
                break
1271
1364
            except errors.NoSuchFile:
1272
1365
                self._reload_or_raise()
 
1366
        self._move_to_front(hit_indices)
1273
1367
 
1274
1368
    def iter_entries_prefix(self, keys):
1275
1369
        """Iterate over keys within the index using prefix matching.
1295
1389
        if not keys:
1296
1390
            return
1297
1391
        seen_keys = set()
 
1392
        hit_indices = []
1298
1393
        while True:
1299
1394
            try:
1300
1395
                for index in self._indices:
 
1396
                    index_hit = False
1301
1397
                    for node in index.iter_entries_prefix(keys):
1302
1398
                        if node[1] in seen_keys:
1303
1399
                            continue
1304
1400
                        seen_keys.add(node[1])
1305
1401
                        yield node
1306
 
                return
 
1402
                        index_hit = True
 
1403
                    if index_hit:
 
1404
                        hit_indices.append(index)
 
1405
                break
1307
1406
            except errors.NoSuchFile:
1308
1407
                self._reload_or_raise()
 
1408
        self._move_to_front(hit_indices)
 
1409
 
 
1410
    def _move_to_front(self, hit_indices):
 
1411
        """Rearrange self._indices so that hit_indices are first.
 
1412
 
 
1413
        Order is maintained as much as possible, e.g. the first unhit index
 
1414
        will be the first index in _indices after the hit_indices, and the
 
1415
        hit_indices will be present in exactly the order they are passed to
 
1416
        _move_to_front.
 
1417
 
 
1418
        _move_to_front propagates to all objects in self._sibling_indices by
 
1419
        calling _move_to_front_by_name.
 
1420
        """
 
1421
        if self._indices[:len(hit_indices)] == hit_indices:
 
1422
            # The 'hit_indices' are already at the front (and in the same
 
1423
            # order), no need to re-order
 
1424
            return
 
1425
        hit_names = self._move_to_front_by_index(hit_indices)
 
1426
        for sibling_idx in self._sibling_indices:
 
1427
            sibling_idx._move_to_front_by_name(hit_names)
 
1428
 
 
1429
    def _move_to_front_by_index(self, hit_indices):
 
1430
        """Core logic for _move_to_front.
 
1431
        
 
1432
        Returns a list of names corresponding to the hit_indices param.
 
1433
        """
 
1434
        indices_info = zip(self._index_names, self._indices)
 
1435
        if 'index' in debug.debug_flags:
 
1436
            mutter('CombinedGraphIndex reordering: currently %r, promoting %r',
 
1437
                   indices_info, hit_indices)
 
1438
        hit_names = []
 
1439
        unhit_names = []
 
1440
        new_hit_indices = []
 
1441
        unhit_indices = []
 
1442
 
 
1443
        for offset, (name, idx) in enumerate(indices_info):
 
1444
            if idx in hit_indices:
 
1445
                hit_names.append(name)
 
1446
                new_hit_indices.append(idx)
 
1447
                if len(new_hit_indices) == len(hit_indices):
 
1448
                    # We've found all of the hit entries, everything else is
 
1449
                    # unhit
 
1450
                    unhit_names.extend(self._index_names[offset+1:])
 
1451
                    unhit_indices.extend(self._indices[offset+1:])
 
1452
                    break
 
1453
            else:
 
1454
                unhit_names.append(name)
 
1455
                unhit_indices.append(idx)
 
1456
 
 
1457
        self._indices = new_hit_indices + unhit_indices
 
1458
        self._index_names = hit_names + unhit_names
 
1459
        if 'index' in debug.debug_flags:
 
1460
            mutter('CombinedGraphIndex reordered: %r', self._indices)
 
1461
        return hit_names
 
1462
 
 
1463
    def _move_to_front_by_name(self, hit_names):
 
1464
        """Moves indices named by 'hit_names' to front of the search order, as
 
1465
        described in _move_to_front.
 
1466
        """
 
1467
        # Translate names to index instances, and then call
 
1468
        # _move_to_front_by_index.
 
1469
        indices_info = zip(self._index_names, self._indices)
 
1470
        hit_indices = []
 
1471
        for name, idx in indices_info:
 
1472
            if name in hit_names:
 
1473
                hit_indices.append(idx)
 
1474
        self._move_to_front_by_index(hit_indices)
 
1475
 
 
1476
    def find_ancestry(self, keys, ref_list_num):
 
1477
        """Find the complete ancestry for the given set of keys.
 
1478
 
 
1479
        Note that this is a whole-ancestry request, so it should be used
 
1480
        sparingly.
 
1481
 
 
1482
        :param keys: An iterable of keys to look for
 
1483
        :param ref_list_num: The reference list which references the parents
 
1484
            we care about.
 
1485
        :return: (parent_map, missing_keys)
 
1486
        """
 
1487
        # XXX: make this call _move_to_front?
 
1488
        missing_keys = set()
 
1489
        parent_map = {}
 
1490
        keys_to_lookup = set(keys)
 
1491
        generation = 0
 
1492
        while keys_to_lookup:
 
1493
            # keys that *all* indexes claim are missing, stop searching them
 
1494
            generation += 1
 
1495
            all_index_missing = None
 
1496
            # print 'gen\tidx\tsub\tn_keys\tn_pmap\tn_miss'
 
1497
            # print '%4d\t\t\t%4d\t%5d\t%5d' % (generation, len(keys_to_lookup),
 
1498
            #                                   len(parent_map),
 
1499
            #                                   len(missing_keys))
 
1500
            for index_idx, index in enumerate(self._indices):
 
1501
                # TODO: we should probably be doing something with
 
1502
                #       'missing_keys' since we've already determined that
 
1503
                #       those revisions have not been found anywhere
 
1504
                index_missing_keys = set()
 
1505
                # Find all of the ancestry we can from this index
 
1506
                # keep looking until the search_keys set is empty, which means
 
1507
                # things we didn't find should be in index_missing_keys
 
1508
                search_keys = keys_to_lookup
 
1509
                sub_generation = 0
 
1510
                # print '    \t%2d\t\t%4d\t%5d\t%5d' % (
 
1511
                #     index_idx, len(search_keys),
 
1512
                #     len(parent_map), len(index_missing_keys))
 
1513
                while search_keys:
 
1514
                    sub_generation += 1
 
1515
                    # TODO: ref_list_num should really be a parameter, since
 
1516
                    #       CombinedGraphIndex does not know what the ref lists
 
1517
                    #       mean.
 
1518
                    search_keys = index._find_ancestors(search_keys,
 
1519
                        ref_list_num, parent_map, index_missing_keys)
 
1520
                    # print '    \t  \t%2d\t%4d\t%5d\t%5d' % (
 
1521
                    #     sub_generation, len(search_keys),
 
1522
                    #     len(parent_map), len(index_missing_keys))
 
1523
                # Now set whatever was missing to be searched in the next index
 
1524
                keys_to_lookup = index_missing_keys
 
1525
                if all_index_missing is None:
 
1526
                    all_index_missing = set(index_missing_keys)
 
1527
                else:
 
1528
                    all_index_missing.intersection_update(index_missing_keys)
 
1529
                if not keys_to_lookup:
 
1530
                    break
 
1531
            if all_index_missing is None:
 
1532
                # There were no indexes, so all search keys are 'missing'
 
1533
                missing_keys.update(keys_to_lookup)
 
1534
                keys_to_lookup = None
 
1535
            else:
 
1536
                missing_keys.update(all_index_missing)
 
1537
                keys_to_lookup.difference_update(all_index_missing)
 
1538
        return parent_map, missing_keys
1309
1539
 
1310
1540
    def key_count(self):
1311
1541
        """Return an estimate of the number of keys in this index.
1340
1570
                         ' Raising original exception.')
1341
1571
            raise exc_type, exc_value, exc_traceback
1342
1572
 
 
1573
    def set_sibling_indices(self, sibling_combined_graph_indices):
 
1574
        """Set the CombinedGraphIndex objects to reorder after reordering self.
 
1575
        """
 
1576
        self._sibling_indices = sibling_combined_graph_indices
 
1577
 
1343
1578
    def validate(self):
1344
1579
        """Validate that everything in the index can be accessed."""
1345
1580
        while True:
1398
1633
            defined order for the result iteration - it will be in the most
1399
1634
            efficient order for the index (keys iteration order in this case).
1400
1635
        """
1401
 
        keys = set(keys)
 
1636
        # Note: See BTreeBuilder.iter_entries for an explanation of why we
 
1637
        #       aren't using set().intersection() here
 
1638
        nodes = self._nodes
 
1639
        keys = [key for key in keys if key in nodes]
1402
1640
        if self.reference_lists:
1403
 
            for key in keys.intersection(self._keys):
1404
 
                node = self._nodes[key]
 
1641
            for key in keys:
 
1642
                node = nodes[key]
1405
1643
                if not node[0]:
1406
1644
                    yield self, key, node[2], node[1]
1407
1645
        else:
1408
 
            for key in keys.intersection(self._keys):
1409
 
                node = self._nodes[key]
 
1646
            for key in keys:
 
1647
                node = nodes[key]
1410
1648
                if not node[0]:
1411
1649
                    yield self, key, node[2]
1412
1650
 
1486
1724
 
1487
1725
        For InMemoryGraphIndex the estimate is exact.
1488
1726
        """
1489
 
        return len(self._keys)
 
1727
        return len(self._nodes) - len(self._absent_keys)
1490
1728
 
1491
1729
    def validate(self):
1492
1730
        """In memory index's have no known corruption at the moment."""