← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: Ian Clatworthy
  • Date: 2010-05-26 04:26:59 UTC
  • mto: (5255.2.1 integration)
  • mto: This revision was merged to the branch mainline in revision 5256.
  • Revision ID: ian.clatworthy@canonical.com-20100526042659-2e3p4qdjr0sby0bt
https://launchpad.net/bugs/486990
Fix PDF generation of User Reference

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
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
104
 
104
105
    def _check_key(self, key):
105
106
        """Raise BadIndexKey if key is not a valid key for this index."""
106
 
        if type(key) != tuple:
 
107
        if type(key) not in (tuple, StaticTuple):
107
108
            raise errors.BadIndexKey(key)
108
109
        if self._key_length != len(key):
109
110
            raise errors.BadIndexKey(key)
165
166
            return
166
167
        key_dict = self._nodes_by_key
167
168
        if self.reference_lists:
168
 
            key_value = key, value, node_refs
 
169
            key_value = StaticTuple(key, value, node_refs)
169
170
        else:
170
 
            key_value = key, value
 
171
            key_value = StaticTuple(key, value)
171
172
        for subkey in key[:-1]:
172
173
            key_dict = key_dict.setdefault(subkey, {})
173
174
        key_dict[key[-1]] = key_value
189
190
                                This may contain duplicates if the same key is
190
191
                                referenced in multiple lists.
191
192
        """
 
193
        as_st = StaticTuple.from_sequence
192
194
        self._check_key(key)
193
195
        if _newline_null_re.search(value) is not None:
194
196
            raise errors.BadIndexValue(value)
203
205
                if reference not in self._nodes:
204
206
                    self._check_key(reference)
205
207
                    absent_references.append(reference)
206
 
            node_refs.append(tuple(reference_list))
207
 
        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
208
212
 
209
213
    def add_node(self, key, value, references=()):
210
214
        """Add a node to the index.
225
229
            # There may be duplicates, but I don't think it is worth worrying
226
230
            # about
227
231
            self._nodes[reference] = ('a', (), '')
 
232
        self._absent_keys.update(absent_references)
 
233
        self._absent_keys.discard(key)
228
234
        self._nodes[key] = ('', node_refs, value)
229
 
        self._keys.add(key)
230
235
        if self._nodes_by_key is not None and self._key_length > 1:
231
236
            self._update_nodes_by_key(key, value, node_refs)
232
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
        
233
245
    def finish(self):
234
246
        lines = [_SIGNATURE]
235
247
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
236
248
        lines.append(_OPTION_KEY_ELEMENTS + str(self._key_length) + '\n')
237
 
        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')
238
251
        prefix_length = sum(len(x) for x in lines)
239
252
        # references are byte offsets. To avoid having to do nasty
240
253
        # polynomial work to resolve offsets (references to later in the
334
347
        if combine_backing_indices is not None:
335
348
            self._combine_backing_indices = combine_backing_indices
336
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
 
365
 
337
366
 
338
367
class GraphIndex(object):
339
368
    """An index for data with embedded graphs.
353
382
    suitable for production use. :XXX
354
383
    """
355
384
 
356
 
    def __init__(self, transport, name, size):
 
385
    def __init__(self, transport, name, size, unlimited_cache=False, offset=0):
357
386
        """Open an index called name on transport.
358
387
 
359
388
        :param transport: A bzrlib.transport.Transport.
365
394
            avoided by having it supplied. If size is None, then bisection
366
395
            support will be disabled and accessing the index will just stream
367
396
            all the data.
 
397
        :param offset: Instead of starting the index data at offset 0, start it
 
398
            at an arbitrary offset.
368
399
        """
369
400
        self._transport = transport
370
401
        self._name = name
387
418
        self._size = size
388
419
        # The number of bytes we've read so far in trying to process this file
389
420
        self._bytes_read = 0
 
421
        self._base_offset = offset
390
422
 
391
423
    def __eq__(self, other):
392
424
        """Equal when self and other were created with the same parameters."""
415
447
            mutter('Reading entire index %s', self._transport.abspath(self._name))
416
448
        if stream is None:
417
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:])
418
454
        self._read_prefix(stream)
419
455
        self._expected_elements = 3 + self._key_length
420
456
        line_count = 0
426
462
        trailers = 0
427
463
        pos = stream.tell()
428
464
        lines = stream.read().split('\n')
 
465
        stream.close()
429
466
        del lines[-1]
430
467
        _, _, _, trailers = self._parse_lines(lines, pos)
431
468
        for key, absent, references, value in self._keys_by_offset.itervalues():
438
475
                node_value = value
439
476
            self._nodes[key] = node_value
440
477
        # cache the keys for quick set intersections
441
 
        self._keys = set(self._nodes)
442
478
        if trailers != 1:
443
479
            # there must be one line - the empty trailer line.
444
480
            raise errors.BadIndexData(self)
445
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
 
446
490
    def external_references(self, ref_list_num):
447
491
        """Return references that are not present in this index.
448
492
        """
451
495
            raise ValueError('No ref list %d, index has %d ref lists'
452
496
                % (ref_list_num, self.node_ref_lists))
453
497
        refs = set()
454
 
        for key, (value, ref_lists) in self._nodes.iteritems():
 
498
        nodes = self._nodes
 
499
        for key, (value, ref_lists) in nodes.iteritems():
455
500
            ref_list = ref_lists[ref_list_num]
456
 
            refs.update(ref_list)
457
 
        return refs - self._keys
 
501
            refs.update([ref for ref in ref_list if ref not in nodes])
 
502
        return refs
458
503
 
459
504
    def _get_nodes_by_key(self):
460
505
        if self._nodes_by_key is None:
587
632
 
588
633
    def _iter_entries_from_total_buffer(self, keys):
589
634
        """Iterate over keys when the entire index is parsed."""
590
 
        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]
591
639
        if self.node_ref_lists:
592
640
            for key in keys:
593
 
                value, node_refs = self._nodes[key]
 
641
                value, node_refs = nodes[key]
594
642
                yield self, key, value, node_refs
595
643
        else:
596
644
            for key in keys:
597
 
                yield self, key, self._nodes[key]
 
645
                yield self, key, nodes[key]
598
646
 
599
647
    def iter_entries(self, keys):
600
648
        """Iterate over keys within the index.
703
751
                # the last thing looked up was a terminal element
704
752
                yield (self, ) + key_dict
705
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
 
706
771
    def key_count(self):
707
772
        """Return an estimate of the number of keys in this index.
708
773
 
1120
1185
            self._parsed_key_map.insert(index + 1, new_key)
1121
1186
 
1122
1187
    def _read_and_parse(self, readv_ranges):
1123
 
        """Read the the ranges and parse the resulting data.
 
1188
        """Read the ranges and parse the resulting data.
1124
1189
 
1125
1190
        :param readv_ranges: A prepared readv range list.
1126
1191
        """
1132
1197
            self._buffer_all()
1133
1198
            return
1134
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]
1135
1205
        readv_data = self._transport.readv(self._name, readv_ranges, True,
1136
 
            self._size)
 
1206
            self._size + self._base_offset)
1137
1207
        # parse
1138
1208
        for offset, data in readv_data:
 
1209
            offset -= base_offset
1139
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
1140
1216
            if offset == 0 and len(data) == self._size:
1141
1217
                # We read the whole range, most likely because the
1142
1218
                # Transport upcast our readv ranges into one long request
1169
1245
    static data.
1170
1246
 
1171
1247
    Queries against the combined index will be made against the first index,
1172
 
    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
1173
1249
    performance significantly. For example, if one index is on local disk and a
1174
1250
    second on a remote server, the local disk index should be before the other
1175
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).
1176
1257
    """
1177
1258
 
1178
1259
    def __init__(self, indices, reload_func=None):
1185
1266
        """
1186
1267
        self._indices = indices
1187
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)
1188
1276
 
1189
1277
    def __repr__(self):
1190
1278
        return "%s(%s)" % (
1191
1279
                self.__class__.__name__,
1192
1280
                ', '.join(map(repr, self._indices)))
1193
1281
 
 
1282
    def clear_cache(self):
 
1283
        """See GraphIndex.clear_cache()"""
 
1284
        for index in self._indices:
 
1285
            index.clear_cache()
 
1286
 
1194
1287
    def get_parent_map(self, keys):
1195
 
        """See graph._StackedParentsProvider.get_parent_map"""
 
1288
        """See graph.StackedParentsProvider.get_parent_map"""
1196
1289
        search_keys = set(keys)
1197
1290
        if NULL_REVISION in search_keys:
1198
1291
            search_keys.discard(NULL_REVISION)
1208
1301
 
1209
1302
    has_key = _has_key_from_parent_map
1210
1303
 
1211
 
    def insert_index(self, pos, index):
 
1304
    def insert_index(self, pos, index, name=None):
1212
1305
        """Insert a new index in the list of indices to query.
1213
1306
 
1214
1307
        :param pos: The position to insert the index.
1215
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)
1216
1312
        """
1217
1313
        self._indices.insert(pos, index)
 
1314
        self._index_names.insert(pos, name)
1218
1315
 
1219
1316
    def iter_all_entries(self):
1220
1317
        """Iterate over all keys within the index
1245
1342
        value and are only reported once.
1246
1343
 
1247
1344
        :param keys: An iterable providing the keys to be retrieved.
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
 
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
1250
1347
            efficient order for the index.
1251
1348
        """
1252
1349
        keys = set(keys)
 
1350
        hit_indices = []
1253
1351
        while True:
1254
1352
            try:
1255
1353
                for index in self._indices:
1256
1354
                    if not keys:
1257
 
                        return
 
1355
                        break
 
1356
                    index_hit = False
1258
1357
                    for node in index.iter_entries(keys):
1259
1358
                        keys.remove(node[1])
1260
1359
                        yield node
1261
 
                return
 
1360
                        index_hit = True
 
1361
                    if index_hit:
 
1362
                        hit_indices.append(index)
 
1363
                break
1262
1364
            except errors.NoSuchFile:
1263
1365
                self._reload_or_raise()
 
1366
        self._move_to_front(hit_indices)
1264
1367
 
1265
1368
    def iter_entries_prefix(self, keys):
1266
1369
        """Iterate over keys within the index using prefix matching.
1286
1389
        if not keys:
1287
1390
            return
1288
1391
        seen_keys = set()
 
1392
        hit_indices = []
1289
1393
        while True:
1290
1394
            try:
1291
1395
                for index in self._indices:
 
1396
                    index_hit = False
1292
1397
                    for node in index.iter_entries_prefix(keys):
1293
1398
                        if node[1] in seen_keys:
1294
1399
                            continue
1295
1400
                        seen_keys.add(node[1])
1296
1401
                        yield node
1297
 
                return
 
1402
                        index_hit = True
 
1403
                    if index_hit:
 
1404
                        hit_indices.append(index)
 
1405
                break
1298
1406
            except errors.NoSuchFile:
1299
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
1300
1539
 
1301
1540
    def key_count(self):
1302
1541
        """Return an estimate of the number of keys in this index.
1331
1570
                         ' Raising original exception.')
1332
1571
            raise exc_type, exc_value, exc_traceback
1333
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
 
1334
1578
    def validate(self):
1335
1579
        """Validate that everything in the index can be accessed."""
1336
1580
        while True:
1389
1633
            defined order for the result iteration - it will be in the most
1390
1634
            efficient order for the index (keys iteration order in this case).
1391
1635
        """
1392
 
        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]
1393
1640
        if self.reference_lists:
1394
 
            for key in keys.intersection(self._keys):
1395
 
                node = self._nodes[key]
 
1641
            for key in keys:
 
1642
                node = nodes[key]
1396
1643
                if not node[0]:
1397
1644
                    yield self, key, node[2], node[1]
1398
1645
        else:
1399
 
            for key in keys.intersection(self._keys):
1400
 
                node = self._nodes[key]
 
1646
            for key in keys:
 
1647
                node = nodes[key]
1401
1648
                if not node[0]:
1402
1649
                    yield self, key, node[2]
1403
1650
 
1477
1724
 
1478
1725
        For InMemoryGraphIndex the estimate is exact.
1479
1726
        """
1480
 
        return len(self._keys)
 
1727
        return len(self._nodes) - len(self._absent_keys)
1481
1728
 
1482
1729
    def validate(self):
1483
1730
        """In memory index's have no known corruption at the moment."""