~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: Aaron Bentley
  • Date: 2008-10-16 19:04:48 UTC
  • mto: (3363.19.4 fix-iter-changes)
  • mto: This revision was merged to the branch mainline in revision 3781.
  • Revision ID: aaron@aaronbentley.com-20081016190448-2a4ilw7bqsw08190
Updates from review (and a doc update)

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2007-2010 Canonical Ltd
 
1
# Copyright (C) 2007 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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""Indexing facilities."""
18
18
 
27
27
from bisect import bisect_right
28
28
from cStringIO import StringIO
29
29
import re
30
 
import sys
31
30
 
32
31
from bzrlib.lazy_import import lazy_import
33
32
lazy_import(globals(), """
39
38
from bzrlib import (
40
39
    debug,
41
40
    errors,
 
41
    symbol_versioning,
42
42
    )
43
 
from bzrlib.static_tuple import StaticTuple
44
43
 
45
44
_HEADER_READV = (0, 200)
46
45
_OPTION_KEY_ELEMENTS = "key_elements="
53
52
_newline_null_re = re.compile('[\n\0]')
54
53
 
55
54
 
56
 
def _has_key_from_parent_map(self, key):
57
 
    """Check if this index has one key.
58
 
 
59
 
    If it's possible to check for multiple keys at once through
60
 
    calling get_parent_map that should be faster.
61
 
    """
62
 
    return (key in self.get_parent_map([key]))
63
 
 
64
 
 
65
 
def _missing_keys_from_parent_map(self, keys):
66
 
    return set(keys) - set(self.get_parent_map(keys))
67
 
 
68
 
 
69
55
class GraphIndexBuilder(object):
70
56
    """A builder that can build a GraphIndex.
71
 
 
 
57
    
72
58
    The resulting graph has the structure:
73
 
 
 
59
    
74
60
    _SIGNATURE OPTIONS NODES NEWLINE
75
61
    _SIGNATURE     := 'Bazaar Graph Index 1' NEWLINE
76
62
    OPTIONS        := 'node_ref_lists=' DIGITS NEWLINE
93
79
        :param key_elements: The number of bytestrings in each key.
94
80
        """
95
81
        self.reference_lists = reference_lists
 
82
        self._keys = set()
96
83
        # A dict of {key: (absent, ref_lists, value)}
97
84
        self._nodes = {}
98
 
        # Keys that are referenced but not actually present in this index
99
 
        self._absent_keys = set()
100
85
        self._nodes_by_key = None
101
86
        self._key_length = key_elements
102
 
        self._optimize_for_size = False
103
 
        self._combine_backing_indices = True
104
87
 
105
88
    def _check_key(self, key):
106
89
        """Raise BadIndexKey if key is not a valid key for this index."""
107
 
        if type(key) not in (tuple, StaticTuple):
 
90
        if type(key) != tuple:
108
91
            raise errors.BadIndexKey(key)
109
92
        if self._key_length != len(key):
110
93
            raise errors.BadIndexKey(key)
112
95
            if not element or _whitespace_re.search(element) is not None:
113
96
                raise errors.BadIndexKey(element)
114
97
 
115
 
    def _external_references(self):
116
 
        """Return references that are not present in this index.
117
 
        """
118
 
        keys = set()
119
 
        refs = set()
120
 
        # TODO: JAM 2008-11-21 This makes an assumption about how the reference
121
 
        #       lists are used. It is currently correct for pack-0.92 through
122
 
        #       1.9, which use the node references (3rd column) second
123
 
        #       reference list as the compression parent. Perhaps this should
124
 
        #       be moved into something higher up the stack, since it
125
 
        #       makes assumptions about how the index is used.
126
 
        if self.reference_lists > 1:
127
 
            for node in self.iter_all_entries():
128
 
                keys.add(node[1])
129
 
                refs.update(node[3][1])
130
 
            return refs - keys
131
 
        else:
132
 
            # If reference_lists == 0 there can be no external references, and
133
 
            # if reference_lists == 1, then there isn't a place to store the
134
 
            # compression parent
135
 
            return set()
136
 
 
137
98
    def _get_nodes_by_key(self):
138
99
        if self._nodes_by_key is None:
139
100
            nodes_by_key = {}
166
127
            return
167
128
        key_dict = self._nodes_by_key
168
129
        if self.reference_lists:
169
 
            key_value = StaticTuple(key, value, node_refs)
 
130
            key_value = key, value, node_refs
170
131
        else:
171
 
            key_value = StaticTuple(key, value)
 
132
            key_value = key, value
172
133
        for subkey in key[:-1]:
173
134
            key_dict = key_dict.setdefault(subkey, {})
174
135
        key_dict[key[-1]] = key_value
190
151
                                This may contain duplicates if the same key is
191
152
                                referenced in multiple lists.
192
153
        """
193
 
        as_st = StaticTuple.from_sequence
194
154
        self._check_key(key)
195
155
        if _newline_null_re.search(value) is not None:
196
156
            raise errors.BadIndexValue(value)
205
165
                if reference not in self._nodes:
206
166
                    self._check_key(reference)
207
167
                    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
 
168
            node_refs.append(tuple(reference_list))
 
169
        return tuple(node_refs), absent_references
212
170
 
213
171
    def add_node(self, key, value, references=()):
214
172
        """Add a node to the index.
229
187
            # There may be duplicates, but I don't think it is worth worrying
230
188
            # about
231
189
            self._nodes[reference] = ('a', (), '')
232
 
        self._absent_keys.update(absent_references)
233
 
        self._absent_keys.discard(key)
234
190
        self._nodes[key] = ('', node_refs, value)
 
191
        self._keys.add(key)
235
192
        if self._nodes_by_key is not None and self._key_length > 1:
236
193
            self._update_nodes_by_key(key, value, node_refs)
237
194
 
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
195
    def finish(self):
246
196
        lines = [_SIGNATURE]
247
197
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
248
198
        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')
 
199
        lines.append(_OPTION_LEN + str(len(self._keys)) + '\n')
251
200
        prefix_length = sum(len(x) for x in lines)
252
201
        # references are byte offsets. To avoid having to do nasty
253
202
        # polynomial work to resolve offsets (references to later in the
260
209
        # one to pad all the data with reference-length and determine entry
261
210
        # addresses.
262
211
        # One to serialise.
263
 
 
 
212
        
264
213
        # forward sorted by key. In future we may consider topological sorting,
265
214
        # at the cost of table scans for direct lookup, or a second index for
266
215
        # direct lookup
329
278
                (len(result.getvalue()), expected_bytes))
330
279
        return result
331
280
 
332
 
    def set_optimize(self, for_size=None, combine_backing_indices=None):
333
 
        """Change how the builder tries to optimize the result.
334
 
 
335
 
        :param for_size: Tell the builder to try and make the index as small as
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.)
341
 
        :return: None
342
 
        """
343
 
        # GraphIndexBuilder itself doesn't pay attention to the flag yet, but
344
 
        # other builders do.
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
365
 
 
366
281
 
367
282
class GraphIndex(object):
368
283
    """An index for data with embedded graphs.
369
 
 
 
284
 
370
285
    The index maps keys to a list of key reference lists, and a value.
371
286
    Each node has the same number of key reference lists. Each key reference
372
287
    list can be empty or an arbitrary length. The value is an opaque NULL
373
 
    terminated string without any newlines. The storage of the index is
 
288
    terminated string without any newlines. The storage of the index is 
374
289
    hidden in the interface: keys and key references are always tuples of
375
290
    bytestrings, never the internal representation (e.g. dictionary offsets).
376
291
 
382
297
    suitable for production use. :XXX
383
298
    """
384
299
 
385
 
    def __init__(self, transport, name, size, unlimited_cache=False):
 
300
    def __init__(self, transport, name, size):
386
301
        """Open an index called name on transport.
387
302
 
388
303
        :param transport: A bzrlib.transport.Transport.
455
370
        trailers = 0
456
371
        pos = stream.tell()
457
372
        lines = stream.read().split('\n')
458
 
        stream.close()
459
373
        del lines[-1]
460
374
        _, _, _, trailers = self._parse_lines(lines, pos)
461
375
        for key, absent, references, value in self._keys_by_offset.itervalues():
468
382
                node_value = value
469
383
            self._nodes[key] = node_value
470
384
        # cache the keys for quick set intersections
 
385
        self._keys = set(self._nodes)
471
386
        if trailers != 1:
472
387
            # there must be one line - the empty trailer line.
473
388
            raise errors.BadIndexData(self)
474
389
 
475
 
    def clear_cache(self):
476
 
        """Clear out any cached/memoized values.
477
 
 
478
 
        This can be called at any time, but generally it is used when we have
479
 
        extracted some information, but don't expect to be requesting any more
480
 
        from this index.
481
 
        """
482
 
 
483
 
    def external_references(self, ref_list_num):
484
 
        """Return references that are not present in this index.
485
 
        """
486
 
        self._buffer_all()
487
 
        if ref_list_num + 1 > self.node_ref_lists:
488
 
            raise ValueError('No ref list %d, index has %d ref lists'
489
 
                % (ref_list_num, self.node_ref_lists))
490
 
        refs = set()
491
 
        nodes = self._nodes
492
 
        for key, (value, ref_lists) in nodes.iteritems():
493
 
            ref_list = ref_lists[ref_list_num]
494
 
            refs.update([ref for ref in ref_list if ref not in nodes])
495
 
        return refs
496
 
 
497
390
    def _get_nodes_by_key(self):
498
391
        if self._nodes_by_key is None:
499
392
            nodes_by_key = {}
561
454
 
562
455
    def _resolve_references(self, references):
563
456
        """Return the resolved key references for references.
564
 
 
 
457
        
565
458
        References are resolved by looking up the location of the key in the
566
459
        _keys_by_offset map and substituting the key name, preserving ordering.
567
460
 
568
 
        :param references: An iterable of iterables of key locations. e.g.
 
461
        :param references: An iterable of iterables of key locations. e.g. 
569
462
            [[123, 456], [123]]
570
463
        :return: A tuple of tuples of keys.
571
464
        """
625
518
 
626
519
    def _iter_entries_from_total_buffer(self, keys):
627
520
        """Iterate over keys when the entire index is parsed."""
628
 
        # Note: See the note in BTreeBuilder.iter_entries for why we don't use
629
 
        #       .intersection() here
630
 
        nodes = self._nodes
631
 
        keys = [key for key in keys if key in nodes]
 
521
        keys = keys.intersection(self._keys)
632
522
        if self.node_ref_lists:
633
523
            for key in keys:
634
 
                value, node_refs = nodes[key]
 
524
                value, node_refs = self._nodes[key]
635
525
                yield self, key, value, node_refs
636
526
        else:
637
527
            for key in keys:
638
 
                yield self, key, nodes[key]
 
528
                yield self, key, self._nodes[key]
639
529
 
640
530
    def iter_entries(self, keys):
641
531
        """Iterate over keys within the index.
732
622
                    # can't be empty or would not exist
733
623
                    item, value = key_dict.iteritems().next()
734
624
                    if type(value) == dict:
735
 
                        # push keys
 
625
                        # push keys 
736
626
                        dicts.extend(key_dict.itervalues())
737
627
                    else:
738
628
                        # yield keys
744
634
                # the last thing looked up was a terminal element
745
635
                yield (self, ) + key_dict
746
636
 
747
 
    def _find_ancestors(self, keys, ref_list_num, parent_map, missing_keys):
748
 
        """See BTreeIndex._find_ancestors."""
749
 
        # The api can be implemented as a trivial overlay on top of
750
 
        # iter_entries, it is not an efficient implementation, but it at least
751
 
        # gets the job done.
752
 
        found_keys = set()
753
 
        search_keys = set()
754
 
        for index, key, value, refs in self.iter_entries(keys):
755
 
            parent_keys = refs[ref_list_num]
756
 
            found_keys.add(key)
757
 
            parent_map[key] = parent_keys
758
 
            search_keys.update(parent_keys)
759
 
        # Figure out what, if anything, was missing
760
 
        missing_keys.update(set(keys).difference(found_keys))
761
 
        search_keys = search_keys.difference(parent_map)
762
 
        return search_keys
763
 
 
764
637
    def key_count(self):
765
638
        """Return an estimate of the number of keys in this index.
766
 
 
 
639
        
767
640
        For GraphIndex the estimate is exact.
768
641
        """
769
642
        if self._key_count is None:
785
658
        # Possible improvements:
786
659
        #  - only bisect lookup each key once
787
660
        #  - sort the keys first, and use that to reduce the bisection window
788
 
        # -----
 
661
        # ----- 
789
662
        # this progresses in three parts:
790
663
        # read data
791
664
        # parse it
800
673
                # We have the key parsed.
801
674
                continue
802
675
            index = self._parsed_key_index(key)
803
 
            if (len(self._parsed_key_map) and
 
676
            if (len(self._parsed_key_map) and 
804
677
                self._parsed_key_map[index][0] <= key and
805
678
                (self._parsed_key_map[index][1] >= key or
806
679
                 # end of the file has been parsed
810
683
                continue
811
684
            # - if we have examined this part of the file already - yes
812
685
            index = self._parsed_byte_index(location)
813
 
            if (len(self._parsed_byte_map) and
 
686
            if (len(self._parsed_byte_map) and 
814
687
                self._parsed_byte_map[index][0] <= location and
815
688
                self._parsed_byte_map[index][1] > location):
816
689
                # the byte region has been parsed, so no read is needed.
1071
944
        # adjust offset and data to the parseable data.
1072
945
        trimmed_data = data[trim_start:trim_end]
1073
946
        if not (trimmed_data):
1074
 
            raise AssertionError('read unneeded data [%d:%d] from [%d:%d]'
 
947
            raise AssertionError('read unneeded data [%d:%d] from [%d:%d]' 
1075
948
                % (trim_start, trim_end, offset, offset + len(data)))
1076
949
        if trim_start:
1077
950
            offset += trim_start
1178
1051
            self._parsed_key_map.insert(index + 1, new_key)
1179
1052
 
1180
1053
    def _read_and_parse(self, readv_ranges):
1181
 
        """Read the ranges and parse the resulting data.
 
1054
        """Read the the ranges and parse the resulting data.
1182
1055
 
1183
1056
        :param readv_ranges: A prepared readv range list.
1184
1057
        """
1222
1095
 
1223
1096
class CombinedGraphIndex(object):
1224
1097
    """A GraphIndex made up from smaller GraphIndices.
1225
 
 
 
1098
    
1226
1099
    The backing indices must implement GraphIndex, and are presumed to be
1227
1100
    static data.
1228
1101
 
1233
1106
    in the index list.
1234
1107
    """
1235
1108
 
1236
 
    def __init__(self, indices, reload_func=None):
 
1109
    def __init__(self, indices):
1237
1110
        """Create a CombinedGraphIndex backed by indices.
1238
1111
 
1239
1112
        :param indices: An ordered list of indices to query for data.
1240
 
        :param reload_func: A function to call if we find we are missing an
1241
 
            index. Should have the form reload_func() => True/False to indicate
1242
 
            if reloading actually changed anything.
1243
1113
        """
1244
1114
        self._indices = indices
1245
 
        self._reload_func = reload_func
1246
1115
 
1247
1116
    def __repr__(self):
1248
1117
        return "%s(%s)" % (
1249
1118
                self.__class__.__name__,
1250
1119
                ', '.join(map(repr, self._indices)))
1251
1120
 
1252
 
    def clear_cache(self):
1253
 
        """See GraphIndex.clear_cache()"""
1254
 
        for index in self._indices:
1255
 
            index.clear_cache()
 
1121
    @symbol_versioning.deprecated_method(symbol_versioning.one_one)
 
1122
    def get_parents(self, revision_ids):
 
1123
        """See graph._StackedParentsProvider.get_parents.
 
1124
        
 
1125
        This implementation thunks the graph.Graph.get_parents api across to
 
1126
        GraphIndex.
 
1127
 
 
1128
        :param revision_ids: An iterable of graph keys for this graph.
 
1129
        :return: A list of parent details for each key in revision_ids.
 
1130
            Each parent details will be one of:
 
1131
             * None when the key was missing
 
1132
             * (NULL_REVISION,) when the key has no parents.
 
1133
             * (parent_key, parent_key...) otherwise.
 
1134
        """
 
1135
        parent_map = self.get_parent_map(revision_ids)
 
1136
        return [parent_map.get(r, None) for r in revision_ids]
1256
1137
 
1257
1138
    def get_parent_map(self, keys):
1258
 
        """See graph.StackedParentsProvider.get_parent_map"""
 
1139
        """See graph._StackedParentsProvider.get_parent_map"""
1259
1140
        search_keys = set(keys)
1260
1141
        if NULL_REVISION in search_keys:
1261
1142
            search_keys.discard(NULL_REVISION)
1269
1150
            found_parents[key] = parents
1270
1151
        return found_parents
1271
1152
 
1272
 
    has_key = _has_key_from_parent_map
1273
 
 
1274
1153
    def insert_index(self, pos, index):
1275
1154
        """Insert a new index in the list of indices to query.
1276
1155
 
1290
1169
            the most efficient order for the index.
1291
1170
        """
1292
1171
        seen_keys = set()
1293
 
        while True:
1294
 
            try:
1295
 
                for index in self._indices:
1296
 
                    for node in index.iter_all_entries():
1297
 
                        if node[1] not in seen_keys:
1298
 
                            yield node
1299
 
                            seen_keys.add(node[1])
1300
 
                return
1301
 
            except errors.NoSuchFile:
1302
 
                self._reload_or_raise()
 
1172
        for index in self._indices:
 
1173
            for node in index.iter_all_entries():
 
1174
                if node[1] not in seen_keys:
 
1175
                    yield node
 
1176
                    seen_keys.add(node[1])
1303
1177
 
1304
1178
    def iter_entries(self, keys):
1305
1179
        """Iterate over keys within the index.
1313
1187
            efficient order for the index.
1314
1188
        """
1315
1189
        keys = set(keys)
1316
 
        while True:
1317
 
            try:
1318
 
                for index in self._indices:
1319
 
                    if not keys:
1320
 
                        return
1321
 
                    for node in index.iter_entries(keys):
1322
 
                        keys.remove(node[1])
1323
 
                        yield node
 
1190
        for index in self._indices:
 
1191
            if not keys:
1324
1192
                return
1325
 
            except errors.NoSuchFile:
1326
 
                self._reload_or_raise()
 
1193
            for node in index.iter_entries(keys):
 
1194
                keys.remove(node[1])
 
1195
                yield node
1327
1196
 
1328
1197
    def iter_entries_prefix(self, keys):
1329
1198
        """Iterate over keys within the index using prefix matching.
1349
1218
        if not keys:
1350
1219
            return
1351
1220
        seen_keys = set()
1352
 
        while True:
1353
 
            try:
1354
 
                for index in self._indices:
1355
 
                    for node in index.iter_entries_prefix(keys):
1356
 
                        if node[1] in seen_keys:
1357
 
                            continue
1358
 
                        seen_keys.add(node[1])
1359
 
                        yield node
1360
 
                return
1361
 
            except errors.NoSuchFile:
1362
 
                self._reload_or_raise()
1363
 
 
1364
 
    def find_ancestry(self, keys, ref_list_num):
1365
 
        """Find the complete ancestry for the given set of keys.
1366
 
 
1367
 
        Note that this is a whole-ancestry request, so it should be used
1368
 
        sparingly.
1369
 
 
1370
 
        :param keys: An iterable of keys to look for
1371
 
        :param ref_list_num: The reference list which references the parents
1372
 
            we care about.
1373
 
        :return: (parent_map, missing_keys)
1374
 
        """
1375
 
        missing_keys = set()
1376
 
        parent_map = {}
1377
 
        keys_to_lookup = set(keys)
1378
 
        generation = 0
1379
 
        while keys_to_lookup:
1380
 
            # keys that *all* indexes claim are missing, stop searching them
1381
 
            generation += 1
1382
 
            all_index_missing = None
1383
 
            # print 'gen\tidx\tsub\tn_keys\tn_pmap\tn_miss'
1384
 
            # print '%4d\t\t\t%4d\t%5d\t%5d' % (generation, len(keys_to_lookup),
1385
 
            #                                   len(parent_map),
1386
 
            #                                   len(missing_keys))
1387
 
            for index_idx, index in enumerate(self._indices):
1388
 
                # TODO: we should probably be doing something with
1389
 
                #       'missing_keys' since we've already determined that
1390
 
                #       those revisions have not been found anywhere
1391
 
                index_missing_keys = set()
1392
 
                # Find all of the ancestry we can from this index
1393
 
                # keep looking until the search_keys set is empty, which means
1394
 
                # things we didn't find should be in index_missing_keys
1395
 
                search_keys = keys_to_lookup
1396
 
                sub_generation = 0
1397
 
                # print '    \t%2d\t\t%4d\t%5d\t%5d' % (
1398
 
                #     index_idx, len(search_keys),
1399
 
                #     len(parent_map), len(index_missing_keys))
1400
 
                while search_keys:
1401
 
                    sub_generation += 1
1402
 
                    # TODO: ref_list_num should really be a parameter, since
1403
 
                    #       CombinedGraphIndex does not know what the ref lists
1404
 
                    #       mean.
1405
 
                    search_keys = index._find_ancestors(search_keys,
1406
 
                        ref_list_num, parent_map, index_missing_keys)
1407
 
                    # print '    \t  \t%2d\t%4d\t%5d\t%5d' % (
1408
 
                    #     sub_generation, len(search_keys),
1409
 
                    #     len(parent_map), len(index_missing_keys))
1410
 
                # Now set whatever was missing to be searched in the next index
1411
 
                keys_to_lookup = index_missing_keys
1412
 
                if all_index_missing is None:
1413
 
                    all_index_missing = set(index_missing_keys)
1414
 
                else:
1415
 
                    all_index_missing.intersection_update(index_missing_keys)
1416
 
                if not keys_to_lookup:
1417
 
                    break
1418
 
            if all_index_missing is None:
1419
 
                # There were no indexes, so all search keys are 'missing'
1420
 
                missing_keys.update(keys_to_lookup)
1421
 
                keys_to_lookup = None
1422
 
            else:
1423
 
                missing_keys.update(all_index_missing)
1424
 
                keys_to_lookup.difference_update(all_index_missing)
1425
 
        return parent_map, missing_keys
 
1221
        for index in self._indices:
 
1222
            for node in index.iter_entries_prefix(keys):
 
1223
                if node[1] in seen_keys:
 
1224
                    continue
 
1225
                seen_keys.add(node[1])
 
1226
                yield node
1426
1227
 
1427
1228
    def key_count(self):
1428
1229
        """Return an estimate of the number of keys in this index.
1429
 
 
 
1230
        
1430
1231
        For CombinedGraphIndex this is approximated by the sum of the keys of
1431
1232
        the child indices. As child indices may have duplicate keys this can
1432
1233
        have a maximum error of the number of child indices * largest number of
1433
1234
        keys in any index.
1434
1235
        """
1435
 
        while True:
1436
 
            try:
1437
 
                return sum((index.key_count() for index in self._indices), 0)
1438
 
            except errors.NoSuchFile:
1439
 
                self._reload_or_raise()
1440
 
 
1441
 
    missing_keys = _missing_keys_from_parent_map
1442
 
 
1443
 
    def _reload_or_raise(self):
1444
 
        """We just got a NoSuchFile exception.
1445
 
 
1446
 
        Try to reload the indices, if it fails, just raise the current
1447
 
        exception.
1448
 
        """
1449
 
        if self._reload_func is None:
1450
 
            raise
1451
 
        exc_type, exc_value, exc_traceback = sys.exc_info()
1452
 
        trace.mutter('Trying to reload after getting exception: %s',
1453
 
                     exc_value)
1454
 
        if not self._reload_func():
1455
 
            # We tried to reload, but nothing changed, so we fail anyway
1456
 
            trace.mutter('_reload_func indicated nothing has changed.'
1457
 
                         ' Raising original exception.')
1458
 
            raise exc_type, exc_value, exc_traceback
 
1236
        return sum((index.key_count() for index in self._indices), 0)
1459
1237
 
1460
1238
    def validate(self):
1461
1239
        """Validate that everything in the index can be accessed."""
1462
 
        while True:
1463
 
            try:
1464
 
                for index in self._indices:
1465
 
                    index.validate()
1466
 
                return
1467
 
            except errors.NoSuchFile:
1468
 
                self._reload_or_raise()
 
1240
        for index in self._indices:
 
1241
            index.validate()
1469
1242
 
1470
1243
 
1471
1244
class InMemoryGraphIndex(GraphIndexBuilder):
1515
1288
            defined order for the result iteration - it will be in the most
1516
1289
            efficient order for the index (keys iteration order in this case).
1517
1290
        """
1518
 
        # Note: See BTreeBuilder.iter_entries for an explanation of why we
1519
 
        #       aren't using set().intersection() here
1520
 
        nodes = self._nodes
1521
 
        keys = [key for key in keys if key in nodes]
 
1291
        keys = set(keys)
1522
1292
        if self.reference_lists:
1523
 
            for key in keys:
1524
 
                node = nodes[key]
 
1293
            for key in keys.intersection(self._keys):
 
1294
                node = self._nodes[key]
1525
1295
                if not node[0]:
1526
1296
                    yield self, key, node[2], node[1]
1527
1297
        else:
1528
 
            for key in keys:
1529
 
                node = nodes[key]
 
1298
            for key in keys.intersection(self._keys):
 
1299
                node = self._nodes[key]
1530
1300
                if not node[0]:
1531
1301
                    yield self, key, node[2]
1532
1302
 
1561
1331
                    raise errors.BadIndexKey(key)
1562
1332
                node = self._nodes[key]
1563
1333
                if node[0]:
1564
 
                    continue
 
1334
                    continue 
1565
1335
                if self.reference_lists:
1566
1336
                    yield self, key, node[2], node[1]
1567
1337
                else:
1592
1362
                    # can't be empty or would not exist
1593
1363
                    item, value = key_dict.iteritems().next()
1594
1364
                    if type(value) == dict:
1595
 
                        # push keys
 
1365
                        # push keys 
1596
1366
                        dicts.extend(key_dict.itervalues())
1597
1367
                    else:
1598
1368
                        # yield keys
1603
1373
 
1604
1374
    def key_count(self):
1605
1375
        """Return an estimate of the number of keys in this index.
1606
 
 
 
1376
        
1607
1377
        For InMemoryGraphIndex the estimate is exact.
1608
1378
        """
1609
 
        return len(self._nodes) - len(self._absent_keys)
 
1379
        return len(self._keys)
1610
1380
 
1611
1381
    def validate(self):
1612
1382
        """In memory index's have no known corruption at the moment."""
1617
1387
 
1618
1388
    Queries against this will emit queries against the adapted Graph with the
1619
1389
    prefix added, queries for all items use iter_entries_prefix. The returned
1620
 
    nodes will have their keys and node references adjusted to remove the
 
1390
    nodes will have their keys and node references adjusted to remove the 
1621
1391
    prefix. Finally, an add_nodes_callback can be supplied - when called the
1622
1392
    nodes and references being added will have prefix prepended.
1623
1393
    """
1650
1420
                    adjusted_references))
1651
1421
        except ValueError:
1652
1422
            # XXX: TODO add an explicit interface for getting the reference list
1653
 
            # status, to handle this bit of user-friendliness in the API more
 
1423
            # status, to handle this bit of user-friendliness in the API more 
1654
1424
            # explicitly.
1655
1425
            for (key, value) in nodes:
1656
1426
                translated_nodes.append((self.prefix + key, value))
1728
1498
 
1729
1499
    def key_count(self):
1730
1500
        """Return an estimate of the number of keys in this index.
1731
 
 
 
1501
        
1732
1502
        For GraphIndexPrefixAdapter this is relatively expensive - key
1733
1503
        iteration with the prefix is done.
1734
1504
        """