← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: Martin Pool
  • Date: 2010-07-16 15:20:17 UTC
  • mfrom: (5346.3.1 pathnotchild)
  • mto: This revision was merged to the branch mainline in revision 5351.
  • Revision ID: mbp@canonical.com-20100716152017-t4c73h9y1uoih7fb
PathNotChild should not give a traceback.

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
        # GZ 2009-09-20: Should really use a try/finally block to ensure close
 
466
        stream.close()
429
467
        del lines[-1]
430
468
        _, _, _, trailers = self._parse_lines(lines, pos)
431
469
        for key, absent, references, value in self._keys_by_offset.itervalues():
438
476
                node_value = value
439
477
            self._nodes[key] = node_value
440
478
        # cache the keys for quick set intersections
441
 
        self._keys = set(self._nodes)
442
479
        if trailers != 1:
443
480
            # there must be one line - the empty trailer line.
444
481
            raise errors.BadIndexData(self)
445
482
 
 
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
 
446
491
    def external_references(self, ref_list_num):
447
492
        """Return references that are not present in this index.
448
493
        """
451
496
            raise ValueError('No ref list %d, index has %d ref lists'
452
497
                % (ref_list_num, self.node_ref_lists))
453
498
        refs = set()
454
 
        for key, (value, ref_lists) in self._nodes.iteritems():
 
499
        nodes = self._nodes
 
500
        for key, (value, ref_lists) in nodes.iteritems():
455
501
            ref_list = ref_lists[ref_list_num]
456
 
            refs.update(ref_list)
457
 
        return refs - self._keys
 
502
            refs.update([ref for ref in ref_list if ref not in nodes])
 
503
        return refs
458
504
 
459
505
    def _get_nodes_by_key(self):
460
506
        if self._nodes_by_key is None:
587
633
 
588
634
    def _iter_entries_from_total_buffer(self, keys):
589
635
        """Iterate over keys when the entire index is parsed."""
590
 
        keys = keys.intersection(self._keys)
 
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]
591
640
        if self.node_ref_lists:
592
641
            for key in keys:
593
 
                value, node_refs = self._nodes[key]
 
642
                value, node_refs = nodes[key]
594
643
                yield self, key, value, node_refs
595
644
        else:
596
645
            for key in keys:
597
 
                yield self, key, self._nodes[key]
 
646
                yield self, key, nodes[key]
598
647
 
599
648
    def iter_entries(self, keys):
600
649
        """Iterate over keys within the index.
703
752
                # the last thing looked up was a terminal element
704
753
                yield (self, ) + key_dict
705
754
 
 
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
 
706
772
    def key_count(self):
707
773
        """Return an estimate of the number of keys in this index.
708
774
 
1120
1186
            self._parsed_key_map.insert(index + 1, new_key)
1121
1187
 
1122
1188
    def _read_and_parse(self, readv_ranges):
1123
 
        """Read the the ranges and parse the resulting data.
 
1189
        """Read the ranges and parse the resulting data.
1124
1190
 
1125
1191
        :param readv_ranges: A prepared readv range list.
1126
1192
        """
1132
1198
            self._buffer_all()
1133
1199
            return
1134
1200
 
 
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]
1135
1206
        readv_data = self._transport.readv(self._name, readv_ranges, True,
1136
 
            self._size)
 
1207
            self._size + self._base_offset)
1137
1208
        # parse
1138
1209
        for offset, data in readv_data:
 
1210
            offset -= base_offset
1139
1211
            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
1140
1217
            if offset == 0 and len(data) == self._size:
1141
1218
                # We read the whole range, most likely because the
1142
1219
                # Transport upcast our readv ranges into one long request
1169
1246
    static data.
1170
1247
 
1171
1248
    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
 
1249
    and then the second and so on. The order of indices can thus influence
1173
1250
    performance significantly. For example, if one index is on local disk and a
1174
1251
    second on a remote server, the local disk index should be before the other
1175
1252
    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).
1176
1258
    """
1177
1259
 
1178
1260
    def __init__(self, indices, reload_func=None):
1185
1267
        """
1186
1268
        self._indices = indices
1187
1269
        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)
1188
1277
 
1189
1278
    def __repr__(self):
1190
1279
        return "%s(%s)" % (
1191
1280
                self.__class__.__name__,
1192
1281
                ', '.join(map(repr, self._indices)))
1193
1282
 
 
1283
    def clear_cache(self):
 
1284
        """See GraphIndex.clear_cache()"""
 
1285
        for index in self._indices:
 
1286
            index.clear_cache()
 
1287
 
1194
1288
    def get_parent_map(self, keys):
1195
 
        """See graph._StackedParentsProvider.get_parent_map"""
 
1289
        """See graph.StackedParentsProvider.get_parent_map"""
1196
1290
        search_keys = set(keys)
1197
1291
        if NULL_REVISION in search_keys:
1198
1292
            search_keys.discard(NULL_REVISION)
1208
1302
 
1209
1303
    has_key = _has_key_from_parent_map
1210
1304
 
1211
 
    def insert_index(self, pos, index):
 
1305
    def insert_index(self, pos, index, name=None):
1212
1306
        """Insert a new index in the list of indices to query.
1213
1307
 
1214
1308
        :param pos: The position to insert the index.
1215
1309
        :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)
1216
1313
        """
1217
1314
        self._indices.insert(pos, index)
 
1315
        self._index_names.insert(pos, name)
1218
1316
 
1219
1317
    def iter_all_entries(self):
1220
1318
        """Iterate over all keys within the index
1245
1343
        value and are only reported once.
1246
1344
 
1247
1345
        :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
 
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
1250
1348
            efficient order for the index.
1251
1349
        """
1252
1350
        keys = set(keys)
 
1351
        hit_indices = []
1253
1352
        while True:
1254
1353
            try:
1255
1354
                for index in self._indices:
1256
1355
                    if not keys:
1257
 
                        return
 
1356
                        break
 
1357
                    index_hit = False
1258
1358
                    for node in index.iter_entries(keys):
1259
1359
                        keys.remove(node[1])
1260
1360
                        yield node
1261
 
                return
 
1361
                        index_hit = True
 
1362
                    if index_hit:
 
1363
                        hit_indices.append(index)
 
1364
                break
1262
1365
            except errors.NoSuchFile:
1263
1366
                self._reload_or_raise()
 
1367
        self._move_to_front(hit_indices)
1264
1368
 
1265
1369
    def iter_entries_prefix(self, keys):
1266
1370
        """Iterate over keys within the index using prefix matching.
1286
1390
        if not keys:
1287
1391
            return
1288
1392
        seen_keys = set()
 
1393
        hit_indices = []
1289
1394
        while True:
1290
1395
            try:
1291
1396
                for index in self._indices:
 
1397
                    index_hit = False
1292
1398
                    for node in index.iter_entries_prefix(keys):
1293
1399
                        if node[1] in seen_keys:
1294
1400
                            continue
1295
1401
                        seen_keys.add(node[1])
1296
1402
                        yield node
1297
 
                return
 
1403
                        index_hit = True
 
1404
                    if index_hit:
 
1405
                        hit_indices.append(index)
 
1406
                break
1298
1407
            except errors.NoSuchFile:
1299
1408
                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
1300
1540
 
1301
1541
    def key_count(self):
1302
1542
        """Return an estimate of the number of keys in this index.
1331
1571
                         ' Raising original exception.')
1332
1572
            raise exc_type, exc_value, exc_traceback
1333
1573
 
 
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
 
1334
1579
    def validate(self):
1335
1580
        """Validate that everything in the index can be accessed."""
1336
1581
        while True:
1389
1634
            defined order for the result iteration - it will be in the most
1390
1635
            efficient order for the index (keys iteration order in this case).
1391
1636
        """
1392
 
        keys = set(keys)
 
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]
1393
1641
        if self.reference_lists:
1394
 
            for key in keys.intersection(self._keys):
1395
 
                node = self._nodes[key]
 
1642
            for key in keys:
 
1643
                node = nodes[key]
1396
1644
                if not node[0]:
1397
1645
                    yield self, key, node[2], node[1]
1398
1646
        else:
1399
 
            for key in keys.intersection(self._keys):
1400
 
                node = self._nodes[key]
 
1647
            for key in keys:
 
1648
                node = nodes[key]
1401
1649
                if not node[0]:
1402
1650
                    yield self, key, node[2]
1403
1651
 
1477
1725
 
1478
1726
        For InMemoryGraphIndex the estimate is exact.
1479
1727
        """
1480
 
        return len(self._keys)
 
1728
        return len(self._nodes) - len(self._absent_keys)
1481
1729
 
1482
1730
    def validate(self):
1483
1731
        """In memory index's have no known corruption at the moment."""