~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: John Arbash Meinel
  • Date: 2009-03-30 18:42:38 UTC
  • mto: (3735.2.171 brisbane-core)
  • mto: This revision was merged to the branch mainline in revision 4280.
  • Revision ID: john@arbash-meinel.com-20090330184238-yc8t2i9wr2mjkxiu
For now, we handle the case of a root node being referenced in the unwanted set.
However, it requires buffering, which is something we want to get rid of.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2007-2010 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
39
39
from bzrlib import (
40
40
    debug,
41
41
    errors,
 
42
    symbol_versioning,
42
43
    )
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()
96
97
        # A dict of {key: (absent, ref_lists, value)}
97
98
        self._nodes = {}
98
 
        # Keys that are referenced but not actually present in this index
99
 
        self._absent_keys = set()
100
99
        self._nodes_by_key = None
101
100
        self._key_length = key_elements
102
101
        self._optimize_for_size = False
104
103
 
105
104
    def _check_key(self, key):
106
105
        """Raise BadIndexKey if key is not a valid key for this index."""
107
 
        if type(key) not in (tuple, StaticTuple):
 
106
        if type(key) != tuple:
108
107
            raise errors.BadIndexKey(key)
109
108
        if self._key_length != len(key):
110
109
            raise errors.BadIndexKey(key)
166
165
            return
167
166
        key_dict = self._nodes_by_key
168
167
        if self.reference_lists:
169
 
            key_value = StaticTuple(key, value, node_refs)
 
168
            key_value = key, value, node_refs
170
169
        else:
171
 
            key_value = StaticTuple(key, value)
 
170
            key_value = key, value
172
171
        for subkey in key[:-1]:
173
172
            key_dict = key_dict.setdefault(subkey, {})
174
173
        key_dict[key[-1]] = key_value
190
189
                                This may contain duplicates if the same key is
191
190
                                referenced in multiple lists.
192
191
        """
193
 
        as_st = StaticTuple.from_sequence
194
192
        self._check_key(key)
195
193
        if _newline_null_re.search(value) is not None:
196
194
            raise errors.BadIndexValue(value)
205
203
                if reference not in self._nodes:
206
204
                    self._check_key(reference)
207
205
                    absent_references.append(reference)
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
 
206
            node_refs.append(tuple(reference_list))
 
207
        return tuple(node_refs), absent_references
212
208
 
213
209
    def add_node(self, key, value, references=()):
214
210
        """Add a node to the index.
229
225
            # There may be duplicates, but I don't think it is worth worrying
230
226
            # about
231
227
            self._nodes[reference] = ('a', (), '')
232
 
        self._absent_keys.update(absent_references)
233
 
        self._absent_keys.discard(key)
234
228
        self._nodes[key] = ('', node_refs, value)
 
229
        self._keys.add(key)
235
230
        if self._nodes_by_key is not None and self._key_length > 1:
236
231
            self._update_nodes_by_key(key, value, node_refs)
237
232
 
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
 
        
245
233
    def finish(self):
246
234
        lines = [_SIGNATURE]
247
235
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
248
236
        lines.append(_OPTION_KEY_ELEMENTS + str(self._key_length) + '\n')
249
 
        key_count = len(self._nodes) - len(self._absent_keys)
250
 
        lines.append(_OPTION_LEN + str(key_count) + '\n')
 
237
        lines.append(_OPTION_LEN + str(len(self._keys)) + '\n')
251
238
        prefix_length = sum(len(x) for x in lines)
252
239
        # references are byte offsets. To avoid having to do nasty
253
240
        # polynomial work to resolve offsets (references to later in the
347
334
        if combine_backing_indices is not None:
348
335
            self._combine_backing_indices = combine_backing_indices
349
336
 
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
365
 
 
366
337
 
367
338
class GraphIndex(object):
368
339
    """An index for data with embedded graphs.
382
353
    suitable for production use. :XXX
383
354
    """
384
355
 
385
 
    def __init__(self, transport, name, size, unlimited_cache=False, offset=0):
 
356
    def __init__(self, transport, name, size):
386
357
        """Open an index called name on transport.
387
358
 
388
359
        :param transport: A bzrlib.transport.Transport.
394
365
            avoided by having it supplied. If size is None, then bisection
395
366
            support will be disabled and accessing the index will just stream
396
367
            all the data.
397
 
        :param offset: Instead of starting the index data at offset 0, start it
398
 
            at an arbitrary offset.
399
368
        """
400
369
        self._transport = transport
401
370
        self._name = name
418
387
        self._size = size
419
388
        # The number of bytes we've read so far in trying to process this file
420
389
        self._bytes_read = 0
421
 
        self._base_offset = offset
422
390
 
423
391
    def __eq__(self, other):
424
392
        """Equal when self and other were created with the same parameters."""
447
415
            mutter('Reading entire index %s', self._transport.abspath(self._name))
448
416
        if stream is None:
449
417
            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:])
454
418
        self._read_prefix(stream)
455
419
        self._expected_elements = 3 + self._key_length
456
420
        line_count = 0
462
426
        trailers = 0
463
427
        pos = stream.tell()
464
428
        lines = stream.read().split('\n')
465
 
        # GZ 2009-09-20: Should really use a try/finally block to ensure close
466
 
        stream.close()
467
429
        del lines[-1]
468
430
        _, _, _, trailers = self._parse_lines(lines, pos)
469
431
        for key, absent, references, value in self._keys_by_offset.itervalues():
476
438
                node_value = value
477
439
            self._nodes[key] = node_value
478
440
        # cache the keys for quick set intersections
 
441
        self._keys = set(self._nodes)
479
442
        if trailers != 1:
480
443
            # there must be one line - the empty trailer line.
481
444
            raise errors.BadIndexData(self)
482
445
 
483
 
    def clear_cache(self):
484
 
        """Clear out any cached/memoized values.
485
 
 
486
 
        This can be called at any time, but generally it is used when we have
487
 
        extracted some information, but don't expect to be requesting any more
488
 
        from this index.
489
 
        """
490
 
 
491
446
    def external_references(self, ref_list_num):
492
447
        """Return references that are not present in this index.
493
448
        """
496
451
            raise ValueError('No ref list %d, index has %d ref lists'
497
452
                % (ref_list_num, self.node_ref_lists))
498
453
        refs = set()
499
 
        nodes = self._nodes
500
 
        for key, (value, ref_lists) in nodes.iteritems():
 
454
        for key, (value, ref_lists) in self._nodes.iteritems():
501
455
            ref_list = ref_lists[ref_list_num]
502
 
            refs.update([ref for ref in ref_list if ref not in nodes])
503
 
        return refs
 
456
            refs.update(ref_list)
 
457
        return refs - self._keys
504
458
 
505
459
    def _get_nodes_by_key(self):
506
460
        if self._nodes_by_key is None:
633
587
 
634
588
    def _iter_entries_from_total_buffer(self, keys):
635
589
        """Iterate over keys when the entire index is parsed."""
636
 
        # Note: See the note in BTreeBuilder.iter_entries for why we don't use
637
 
        #       .intersection() here
638
 
        nodes = self._nodes
639
 
        keys = [key for key in keys if key in nodes]
 
590
        keys = keys.intersection(self._keys)
640
591
        if self.node_ref_lists:
641
592
            for key in keys:
642
 
                value, node_refs = nodes[key]
 
593
                value, node_refs = self._nodes[key]
643
594
                yield self, key, value, node_refs
644
595
        else:
645
596
            for key in keys:
646
 
                yield self, key, nodes[key]
 
597
                yield self, key, self._nodes[key]
647
598
 
648
599
    def iter_entries(self, keys):
649
600
        """Iterate over keys within the index.
752
703
                # the last thing looked up was a terminal element
753
704
                yield (self, ) + key_dict
754
705
 
755
 
    def _find_ancestors(self, keys, ref_list_num, parent_map, missing_keys):
756
 
        """See BTreeIndex._find_ancestors."""
757
 
        # The api can be implemented as a trivial overlay on top of
758
 
        # iter_entries, it is not an efficient implementation, but it at least
759
 
        # gets the job done.
760
 
        found_keys = set()
761
 
        search_keys = set()
762
 
        for index, key, value, refs in self.iter_entries(keys):
763
 
            parent_keys = refs[ref_list_num]
764
 
            found_keys.add(key)
765
 
            parent_map[key] = parent_keys
766
 
            search_keys.update(parent_keys)
767
 
        # Figure out what, if anything, was missing
768
 
        missing_keys.update(set(keys).difference(found_keys))
769
 
        search_keys = search_keys.difference(parent_map)
770
 
        return search_keys
771
 
 
772
706
    def key_count(self):
773
707
        """Return an estimate of the number of keys in this index.
774
708
 
1186
1120
            self._parsed_key_map.insert(index + 1, new_key)
1187
1121
 
1188
1122
    def _read_and_parse(self, readv_ranges):
1189
 
        """Read the ranges and parse the resulting data.
 
1123
        """Read the the ranges and parse the resulting data.
1190
1124
 
1191
1125
        :param readv_ranges: A prepared readv range list.
1192
1126
        """
1198
1132
            self._buffer_all()
1199
1133
            return
1200
1134
 
1201
 
        base_offset = self._base_offset
1202
 
        if base_offset != 0:
1203
 
            # Rewrite the ranges for the offset
1204
 
            readv_ranges = [(start+base_offset, size)
1205
 
                            for start, size in readv_ranges]
1206
1135
        readv_data = self._transport.readv(self._name, readv_ranges, True,
1207
 
            self._size + self._base_offset)
 
1136
            self._size)
1208
1137
        # parse
1209
1138
        for offset, data in readv_data:
1210
 
            offset -= base_offset
1211
1139
            self._bytes_read += len(data)
1212
 
            if offset < 0:
1213
 
                # transport.readv() expanded to extra data which isn't part of
1214
 
                # this index
1215
 
                data = data[-offset:]
1216
 
                offset = 0
1217
1140
            if offset == 0 and len(data) == self._size:
1218
1141
                # We read the whole range, most likely because the
1219
1142
                # Transport upcast our readv ranges into one long request
1246
1169
    static data.
1247
1170
 
1248
1171
    Queries against the combined index will be made against the first index,
1249
 
    and then the second and so on. The order of indices can thus influence
 
1172
    and then the second and so on. The order of index's can thus influence
1250
1173
    performance significantly. For example, if one index is on local disk and a
1251
1174
    second on a remote server, the local disk index should be before the other
1252
1175
    in the index list.
1253
 
    
1254
 
    Also, queries tend to need results from the same indices as previous
1255
 
    queries.  So the indices will be reordered after every query to put the
1256
 
    indices that had the result(s) of that query first (while otherwise
1257
 
    preserving the relative ordering).
1258
1176
    """
1259
1177
 
1260
1178
    def __init__(self, indices, reload_func=None):
1267
1185
        """
1268
1186
        self._indices = indices
1269
1187
        self._reload_func = reload_func
1270
 
        # Sibling indices are other CombinedGraphIndex that we should call
1271
 
        # _move_to_front_by_name on when we auto-reorder ourself.
1272
 
        self._sibling_indices = []
1273
 
        # A list of names that corresponds to the instances in self._indices,
1274
 
        # so _index_names[0] is always the name for _indices[0], etc.  Sibling
1275
 
        # indices must all use the same set of names as each other.
1276
 
        self._index_names = [None] * len(self._indices)
1277
1188
 
1278
1189
    def __repr__(self):
1279
1190
        return "%s(%s)" % (
1280
1191
                self.__class__.__name__,
1281
1192
                ', '.join(map(repr, self._indices)))
1282
1193
 
1283
 
    def clear_cache(self):
1284
 
        """See GraphIndex.clear_cache()"""
1285
 
        for index in self._indices:
1286
 
            index.clear_cache()
1287
 
 
1288
1194
    def get_parent_map(self, keys):
1289
 
        """See graph.StackedParentsProvider.get_parent_map"""
 
1195
        """See graph._StackedParentsProvider.get_parent_map"""
1290
1196
        search_keys = set(keys)
1291
1197
        if NULL_REVISION in search_keys:
1292
1198
            search_keys.discard(NULL_REVISION)
1302
1208
 
1303
1209
    has_key = _has_key_from_parent_map
1304
1210
 
1305
 
    def insert_index(self, pos, index, name=None):
 
1211
    def insert_index(self, pos, index):
1306
1212
        """Insert a new index in the list of indices to query.
1307
1213
 
1308
1214
        :param pos: The position to insert the index.
1309
1215
        :param index: The index to insert.
1310
 
        :param name: a name for this index, e.g. a pack name.  These names can
1311
 
            be used to reflect index reorderings to related CombinedGraphIndex
1312
 
            instances that use the same names.  (see set_sibling_indices)
1313
1216
        """
1314
1217
        self._indices.insert(pos, index)
1315
 
        self._index_names.insert(pos, name)
1316
1218
 
1317
1219
    def iter_all_entries(self):
1318
1220
        """Iterate over all keys within the index
1343
1245
        value and are only reported once.
1344
1246
 
1345
1247
        :param keys: An iterable providing the keys to be retrieved.
1346
 
        :return: An iterable of (index, key, reference_lists, value). There is
1347
 
            no defined order for the result iteration - it will be in the most
 
1248
        :return: An iterable of (index, key, reference_lists, value). There is no
 
1249
            defined order for the result iteration - it will be in the most
1348
1250
            efficient order for the index.
1349
1251
        """
1350
1252
        keys = set(keys)
1351
 
        hit_indices = []
1352
1253
        while True:
1353
1254
            try:
1354
1255
                for index in self._indices:
1355
1256
                    if not keys:
1356
 
                        break
1357
 
                    index_hit = False
 
1257
                        return
1358
1258
                    for node in index.iter_entries(keys):
1359
1259
                        keys.remove(node[1])
1360
1260
                        yield node
1361
 
                        index_hit = True
1362
 
                    if index_hit:
1363
 
                        hit_indices.append(index)
1364
 
                break
 
1261
                return
1365
1262
            except errors.NoSuchFile:
1366
1263
                self._reload_or_raise()
1367
 
        self._move_to_front(hit_indices)
1368
1264
 
1369
1265
    def iter_entries_prefix(self, keys):
1370
1266
        """Iterate over keys within the index using prefix matching.
1390
1286
        if not keys:
1391
1287
            return
1392
1288
        seen_keys = set()
1393
 
        hit_indices = []
1394
1289
        while True:
1395
1290
            try:
1396
1291
                for index in self._indices:
1397
 
                    index_hit = False
1398
1292
                    for node in index.iter_entries_prefix(keys):
1399
1293
                        if node[1] in seen_keys:
1400
1294
                            continue
1401
1295
                        seen_keys.add(node[1])
1402
1296
                        yield node
1403
 
                        index_hit = True
1404
 
                    if index_hit:
1405
 
                        hit_indices.append(index)
1406
 
                break
 
1297
                return
1407
1298
            except errors.NoSuchFile:
1408
1299
                self._reload_or_raise()
1409
 
        self._move_to_front(hit_indices)
1410
 
 
1411
 
    def _move_to_front(self, hit_indices):
1412
 
        """Rearrange self._indices so that hit_indices are first.
1413
 
 
1414
 
        Order is maintained as much as possible, e.g. the first unhit index
1415
 
        will be the first index in _indices after the hit_indices, and the
1416
 
        hit_indices will be present in exactly the order they are passed to
1417
 
        _move_to_front.
1418
 
 
1419
 
        _move_to_front propagates to all objects in self._sibling_indices by
1420
 
        calling _move_to_front_by_name.
1421
 
        """
1422
 
        if self._indices[:len(hit_indices)] == hit_indices:
1423
 
            # The 'hit_indices' are already at the front (and in the same
1424
 
            # order), no need to re-order
1425
 
            return
1426
 
        hit_names = self._move_to_front_by_index(hit_indices)
1427
 
        for sibling_idx in self._sibling_indices:
1428
 
            sibling_idx._move_to_front_by_name(hit_names)
1429
 
 
1430
 
    def _move_to_front_by_index(self, hit_indices):
1431
 
        """Core logic for _move_to_front.
1432
 
        
1433
 
        Returns a list of names corresponding to the hit_indices param.
1434
 
        """
1435
 
        indices_info = zip(self._index_names, self._indices)
1436
 
        if 'index' in debug.debug_flags:
1437
 
            mutter('CombinedGraphIndex reordering: currently %r, promoting %r',
1438
 
                   indices_info, hit_indices)
1439
 
        hit_names = []
1440
 
        unhit_names = []
1441
 
        new_hit_indices = []
1442
 
        unhit_indices = []
1443
 
 
1444
 
        for offset, (name, idx) in enumerate(indices_info):
1445
 
            if idx in hit_indices:
1446
 
                hit_names.append(name)
1447
 
                new_hit_indices.append(idx)
1448
 
                if len(new_hit_indices) == len(hit_indices):
1449
 
                    # We've found all of the hit entries, everything else is
1450
 
                    # unhit
1451
 
                    unhit_names.extend(self._index_names[offset+1:])
1452
 
                    unhit_indices.extend(self._indices[offset+1:])
1453
 
                    break
1454
 
            else:
1455
 
                unhit_names.append(name)
1456
 
                unhit_indices.append(idx)
1457
 
 
1458
 
        self._indices = new_hit_indices + unhit_indices
1459
 
        self._index_names = hit_names + unhit_names
1460
 
        if 'index' in debug.debug_flags:
1461
 
            mutter('CombinedGraphIndex reordered: %r', self._indices)
1462
 
        return hit_names
1463
 
 
1464
 
    def _move_to_front_by_name(self, hit_names):
1465
 
        """Moves indices named by 'hit_names' to front of the search order, as
1466
 
        described in _move_to_front.
1467
 
        """
1468
 
        # Translate names to index instances, and then call
1469
 
        # _move_to_front_by_index.
1470
 
        indices_info = zip(self._index_names, self._indices)
1471
 
        hit_indices = []
1472
 
        for name, idx in indices_info:
1473
 
            if name in hit_names:
1474
 
                hit_indices.append(idx)
1475
 
        self._move_to_front_by_index(hit_indices)
1476
 
 
1477
 
    def find_ancestry(self, keys, ref_list_num):
1478
 
        """Find the complete ancestry for the given set of keys.
1479
 
 
1480
 
        Note that this is a whole-ancestry request, so it should be used
1481
 
        sparingly.
1482
 
 
1483
 
        :param keys: An iterable of keys to look for
1484
 
        :param ref_list_num: The reference list which references the parents
1485
 
            we care about.
1486
 
        :return: (parent_map, missing_keys)
1487
 
        """
1488
 
        # XXX: make this call _move_to_front?
1489
 
        missing_keys = set()
1490
 
        parent_map = {}
1491
 
        keys_to_lookup = set(keys)
1492
 
        generation = 0
1493
 
        while keys_to_lookup:
1494
 
            # keys that *all* indexes claim are missing, stop searching them
1495
 
            generation += 1
1496
 
            all_index_missing = None
1497
 
            # print 'gen\tidx\tsub\tn_keys\tn_pmap\tn_miss'
1498
 
            # print '%4d\t\t\t%4d\t%5d\t%5d' % (generation, len(keys_to_lookup),
1499
 
            #                                   len(parent_map),
1500
 
            #                                   len(missing_keys))
1501
 
            for index_idx, index in enumerate(self._indices):
1502
 
                # TODO: we should probably be doing something with
1503
 
                #       'missing_keys' since we've already determined that
1504
 
                #       those revisions have not been found anywhere
1505
 
                index_missing_keys = set()
1506
 
                # Find all of the ancestry we can from this index
1507
 
                # keep looking until the search_keys set is empty, which means
1508
 
                # things we didn't find should be in index_missing_keys
1509
 
                search_keys = keys_to_lookup
1510
 
                sub_generation = 0
1511
 
                # print '    \t%2d\t\t%4d\t%5d\t%5d' % (
1512
 
                #     index_idx, len(search_keys),
1513
 
                #     len(parent_map), len(index_missing_keys))
1514
 
                while search_keys:
1515
 
                    sub_generation += 1
1516
 
                    # TODO: ref_list_num should really be a parameter, since
1517
 
                    #       CombinedGraphIndex does not know what the ref lists
1518
 
                    #       mean.
1519
 
                    search_keys = index._find_ancestors(search_keys,
1520
 
                        ref_list_num, parent_map, index_missing_keys)
1521
 
                    # print '    \t  \t%2d\t%4d\t%5d\t%5d' % (
1522
 
                    #     sub_generation, len(search_keys),
1523
 
                    #     len(parent_map), len(index_missing_keys))
1524
 
                # Now set whatever was missing to be searched in the next index
1525
 
                keys_to_lookup = index_missing_keys
1526
 
                if all_index_missing is None:
1527
 
                    all_index_missing = set(index_missing_keys)
1528
 
                else:
1529
 
                    all_index_missing.intersection_update(index_missing_keys)
1530
 
                if not keys_to_lookup:
1531
 
                    break
1532
 
            if all_index_missing is None:
1533
 
                # There were no indexes, so all search keys are 'missing'
1534
 
                missing_keys.update(keys_to_lookup)
1535
 
                keys_to_lookup = None
1536
 
            else:
1537
 
                missing_keys.update(all_index_missing)
1538
 
                keys_to_lookup.difference_update(all_index_missing)
1539
 
        return parent_map, missing_keys
1540
1300
 
1541
1301
    def key_count(self):
1542
1302
        """Return an estimate of the number of keys in this index.
1571
1331
                         ' Raising original exception.')
1572
1332
            raise exc_type, exc_value, exc_traceback
1573
1333
 
1574
 
    def set_sibling_indices(self, sibling_combined_graph_indices):
1575
 
        """Set the CombinedGraphIndex objects to reorder after reordering self.
1576
 
        """
1577
 
        self._sibling_indices = sibling_combined_graph_indices
1578
 
 
1579
1334
    def validate(self):
1580
1335
        """Validate that everything in the index can be accessed."""
1581
1336
        while True:
1634
1389
            defined order for the result iteration - it will be in the most
1635
1390
            efficient order for the index (keys iteration order in this case).
1636
1391
        """
1637
 
        # Note: See BTreeBuilder.iter_entries for an explanation of why we
1638
 
        #       aren't using set().intersection() here
1639
 
        nodes = self._nodes
1640
 
        keys = [key for key in keys if key in nodes]
 
1392
        keys = set(keys)
1641
1393
        if self.reference_lists:
1642
 
            for key in keys:
1643
 
                node = nodes[key]
 
1394
            for key in keys.intersection(self._keys):
 
1395
                node = self._nodes[key]
1644
1396
                if not node[0]:
1645
1397
                    yield self, key, node[2], node[1]
1646
1398
        else:
1647
 
            for key in keys:
1648
 
                node = nodes[key]
 
1399
            for key in keys.intersection(self._keys):
 
1400
                node = self._nodes[key]
1649
1401
                if not node[0]:
1650
1402
                    yield self, key, node[2]
1651
1403
 
1725
1477
 
1726
1478
        For InMemoryGraphIndex the estimate is exact.
1727
1479
        """
1728
 
        return len(self._nodes) - len(self._absent_keys)
 
1480
        return len(self._keys)
1729
1481
 
1730
1482
    def validate(self):
1731
1483
        """In memory index's have no known corruption at the moment."""