~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: John Arbash Meinel
  • Date: 2007-12-05 19:55:07 UTC
  • mto: This revision was merged to the branch mainline in revision 3082.
  • Revision ID: john@arbash-meinel.com-20071205195507-1ql7vuval5qug5eu
Clean up some vim: lines to make them proper ReST comments.

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(), """
36
35
from bzrlib.revision import NULL_REVISION
37
36
from bzrlib.trace import mutter
38
37
""")
39
 
from bzrlib import (
40
 
    debug,
41
 
    errors,
42
 
    )
43
 
from bzrlib.static_tuple import StaticTuple
 
38
from bzrlib import debug, errors
44
39
 
45
40
_HEADER_READV = (0, 200)
46
41
_OPTION_KEY_ELEMENTS = "key_elements="
53
48
_newline_null_re = re.compile('[\n\0]')
54
49
 
55
50
 
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
51
class GraphIndexBuilder(object):
70
52
    """A builder that can build a GraphIndex.
71
 
 
 
53
    
72
54
    The resulting graph has the structure:
73
 
 
 
55
    
74
56
    _SIGNATURE OPTIONS NODES NEWLINE
75
57
    _SIGNATURE     := 'Bazaar Graph Index 1' NEWLINE
76
58
    OPTIONS        := 'node_ref_lists=' DIGITS NEWLINE
93
75
        :param key_elements: The number of bytestrings in each key.
94
76
        """
95
77
        self.reference_lists = reference_lists
96
 
        # A dict of {key: (absent, ref_lists, value)}
 
78
        self._keys = set()
97
79
        self._nodes = {}
98
 
        # Keys that are referenced but not actually present in this index
99
 
        self._absent_keys = set()
100
 
        self._nodes_by_key = None
 
80
        self._nodes_by_key = {}
101
81
        self._key_length = key_elements
102
 
        self._optimize_for_size = False
103
 
        self._combine_backing_indices = True
104
82
 
105
83
    def _check_key(self, key):
106
84
        """Raise BadIndexKey if key is not a valid key for this index."""
107
 
        if type(key) not in (tuple, StaticTuple):
 
85
        if type(key) != tuple:
108
86
            raise errors.BadIndexKey(key)
109
87
        if self._key_length != len(key):
110
88
            raise errors.BadIndexKey(key)
112
90
            if not element or _whitespace_re.search(element) is not None:
113
91
                raise errors.BadIndexKey(element)
114
92
 
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
 
    def _get_nodes_by_key(self):
138
 
        if self._nodes_by_key is None:
139
 
            nodes_by_key = {}
140
 
            if self.reference_lists:
141
 
                for key, (absent, references, value) in self._nodes.iteritems():
142
 
                    if absent:
143
 
                        continue
144
 
                    key_dict = nodes_by_key
145
 
                    for subkey in key[:-1]:
146
 
                        key_dict = key_dict.setdefault(subkey, {})
147
 
                    key_dict[key[-1]] = key, value, references
148
 
            else:
149
 
                for key, (absent, references, value) in self._nodes.iteritems():
150
 
                    if absent:
151
 
                        continue
152
 
                    key_dict = nodes_by_key
153
 
                    for subkey in key[:-1]:
154
 
                        key_dict = key_dict.setdefault(subkey, {})
155
 
                    key_dict[key[-1]] = key, value
156
 
            self._nodes_by_key = nodes_by_key
157
 
        return self._nodes_by_key
158
 
 
159
 
    def _update_nodes_by_key(self, key, value, node_refs):
160
 
        """Update the _nodes_by_key dict with a new key.
161
 
 
162
 
        For a key of (foo, bar, baz) create
163
 
        _nodes_by_key[foo][bar][baz] = key_value
164
 
        """
165
 
        if self._nodes_by_key is None:
166
 
            return
167
 
        key_dict = self._nodes_by_key
168
 
        if self.reference_lists:
169
 
            key_value = StaticTuple(key, value, node_refs)
170
 
        else:
171
 
            key_value = StaticTuple(key, value)
172
 
        for subkey in key[:-1]:
173
 
            key_dict = key_dict.setdefault(subkey, {})
174
 
        key_dict[key[-1]] = key_value
175
 
 
176
 
    def _check_key_ref_value(self, key, references, value):
177
 
        """Check that 'key' and 'references' are all valid.
178
 
 
179
 
        :param key: A key tuple. Must conform to the key interface (be a tuple,
180
 
            be of the right length, not have any whitespace or nulls in any key
181
 
            element.)
182
 
        :param references: An iterable of reference lists. Something like
183
 
            [[(ref, key)], [(ref, key), (other, key)]]
184
 
        :param value: The value associate with this key. Must not contain
185
 
            newlines or null characters.
186
 
        :return: (node_refs, absent_references)
187
 
            node_refs   basically a packed form of 'references' where all
188
 
                        iterables are tuples
189
 
            absent_references   reference keys that are not in self._nodes.
190
 
                                This may contain duplicates if the same key is
191
 
                                referenced in multiple lists.
192
 
        """
193
 
        as_st = StaticTuple.from_sequence
 
93
    def add_node(self, key, value, references=()):
 
94
        """Add a node to the index.
 
95
 
 
96
        :param key: The key. keys are non-empty tuples containing
 
97
            as many whitespace-free utf8 bytestrings as the key length
 
98
            defined for this index.
 
99
        :param references: An iterable of iterables of keys. Each is a
 
100
            reference to another key.
 
101
        :param value: The value to associate with the key. It may be any
 
102
            bytes as long as it does not contain \0 or \n.
 
103
        """
194
104
        self._check_key(key)
195
105
        if _newline_null_re.search(value) is not None:
196
106
            raise errors.BadIndexValue(value)
197
107
        if len(references) != self.reference_lists:
198
108
            raise errors.BadIndexValue(references)
199
109
        node_refs = []
200
 
        absent_references = []
201
110
        for reference_list in references:
202
111
            for reference in reference_list:
203
 
                # If reference *is* in self._nodes, then we know it has already
204
 
                # been checked.
 
112
                self._check_key(reference)
205
113
                if reference not in self._nodes:
206
 
                    self._check_key(reference)
207
 
                    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
212
 
 
213
 
    def add_node(self, key, value, references=()):
214
 
        """Add a node to the index.
215
 
 
216
 
        :param key: The key. keys are non-empty tuples containing
217
 
            as many whitespace-free utf8 bytestrings as the key length
218
 
            defined for this index.
219
 
        :param references: An iterable of iterables of keys. Each is a
220
 
            reference to another key.
221
 
        :param value: The value to associate with the key. It may be any
222
 
            bytes as long as it does not contain \0 or \n.
223
 
        """
224
 
        (node_refs,
225
 
         absent_references) = self._check_key_ref_value(key, references, value)
226
 
        if key in self._nodes and self._nodes[key][0] != 'a':
 
114
                    self._nodes[reference] = ('a', (), '')
 
115
            node_refs.append(tuple(reference_list))
 
116
        if key in self._nodes and self._nodes[key][0] == '':
227
117
            raise errors.BadIndexDuplicateKey(key, self)
228
 
        for reference in absent_references:
229
 
            # There may be duplicates, but I don't think it is worth worrying
230
 
            # about
231
 
            self._nodes[reference] = ('a', (), '')
232
 
        self._absent_keys.update(absent_references)
233
 
        self._absent_keys.discard(key)
234
 
        self._nodes[key] = ('', node_refs, value)
235
 
        if self._nodes_by_key is not None and self._key_length > 1:
236
 
            self._update_nodes_by_key(key, value, node_refs)
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
 
        
 
118
        self._nodes[key] = ('', tuple(node_refs), value)
 
119
        self._keys.add(key)
 
120
        if self._key_length > 1:
 
121
            key_dict = self._nodes_by_key
 
122
            if self.reference_lists:
 
123
                key_value = key, value, tuple(node_refs)
 
124
            else:
 
125
                key_value = key, value
 
126
            # possibly should do this on-demand, but it seems likely it is 
 
127
            # always wanted
 
128
            # For a key of (foo, bar, baz) create
 
129
            # _nodes_by_key[foo][bar][baz] = key_value
 
130
            for subkey in key[:-1]:
 
131
                key_dict = key_dict.setdefault(subkey, {})
 
132
            key_dict[key[-1]] = key_value
 
133
 
245
134
    def finish(self):
246
135
        lines = [_SIGNATURE]
247
136
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
248
137
        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')
 
138
        lines.append(_OPTION_LEN + str(len(self._keys)) + '\n')
251
139
        prefix_length = sum(len(x) for x in lines)
252
140
        # references are byte offsets. To avoid having to do nasty
253
 
        # polynomial work to resolve offsets (references to later in the
 
141
        # polynomial work to resolve offsets (references to later in the 
254
142
        # file cannot be determined until all the inbetween references have
255
143
        # been calculated too) we pad the offsets with 0's to make them be
256
144
        # of consistent length. Using binary offsets would break the trivial
260
148
        # one to pad all the data with reference-length and determine entry
261
149
        # addresses.
262
150
        # One to serialise.
263
 
 
 
151
        
264
152
        # forward sorted by key. In future we may consider topological sorting,
265
153
        # at the cost of table scans for direct lookup, or a second index for
266
154
        # direct lookup
327
215
            raise errors.BzrError('Failed index creation. Internal error:'
328
216
                ' mismatched output length and expected length: %d %d' %
329
217
                (len(result.getvalue()), expected_bytes))
330
 
        return result
331
 
 
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
 
218
        return StringIO(''.join(lines))
365
219
 
366
220
 
367
221
class GraphIndex(object):
368
222
    """An index for data with embedded graphs.
369
 
 
 
223
 
370
224
    The index maps keys to a list of key reference lists, and a value.
371
225
    Each node has the same number of key reference lists. Each key reference
372
226
    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
 
227
    terminated string without any newlines. The storage of the index is 
374
228
    hidden in the interface: keys and key references are always tuples of
375
229
    bytestrings, never the internal representation (e.g. dictionary offsets).
376
230
 
382
236
    suitable for production use. :XXX
383
237
    """
384
238
 
385
 
    def __init__(self, transport, name, size, unlimited_cache=False, offset=0):
 
239
    def __init__(self, transport, name, size):
386
240
        """Open an index called name on transport.
387
241
 
388
242
        :param transport: A bzrlib.transport.Transport.
394
248
            avoided by having it supplied. If size is None, then bisection
395
249
            support will be disabled and accessing the index will just stream
396
250
            all the data.
397
 
        :param offset: Instead of starting the index data at offset 0, start it
398
 
            at an arbitrary offset.
399
251
        """
400
252
        self._transport = transport
401
253
        self._name = name
416
268
        self._keys_by_offset = None
417
269
        self._nodes_by_key = None
418
270
        self._size = size
419
 
        # The number of bytes we've read so far in trying to process this file
420
 
        self._bytes_read = 0
421
 
        self._base_offset = offset
422
271
 
423
272
    def __eq__(self, other):
424
273
        """Equal when self and other were created with the same parameters."""
431
280
    def __ne__(self, other):
432
281
        return not self.__eq__(other)
433
282
 
434
 
    def __repr__(self):
435
 
        return "%s(%r)" % (self.__class__.__name__,
436
 
            self._transport.abspath(self._name))
437
 
 
438
 
    def _buffer_all(self, stream=None):
 
283
    def _buffer_all(self):
439
284
        """Buffer all the index data.
440
285
 
441
286
        Mutates self._nodes and self.keys_by_offset.
442
287
        """
443
 
        if self._nodes is not None:
444
 
            # We already did this
445
 
            return
446
288
        if 'index' in debug.debug_flags:
447
289
            mutter('Reading entire index %s', self._transport.abspath(self._name))
448
 
        if stream is None:
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:])
 
290
        stream = self._transport.get(self._name)
454
291
        self._read_prefix(stream)
455
292
        self._expected_elements = 3 + self._key_length
456
293
        line_count = 0
458
295
        self._keys_by_offset = {}
459
296
        # ready-to-return key:value or key:value, node_ref_lists
460
297
        self._nodes = {}
461
 
        self._nodes_by_key = None
 
298
        self._nodes_by_key = {}
462
299
        trailers = 0
463
300
        pos = stream.tell()
464
301
        lines = stream.read().split('\n')
465
 
        stream.close()
466
302
        del lines[-1]
467
303
        _, _, _, trailers = self._parse_lines(lines, pos)
468
304
        for key, absent, references, value in self._keys_by_offset.itervalues():
474
310
            else:
475
311
                node_value = value
476
312
            self._nodes[key] = node_value
 
313
            if self._key_length > 1:
 
314
                subkey = list(reversed(key[:-1]))
 
315
                key_dict = self._nodes_by_key
 
316
                if self.node_ref_lists:
 
317
                    key_value = key, node_value[0], node_value[1]
 
318
                else:
 
319
                    key_value = key, node_value
 
320
                # possibly should do this on-demand, but it seems likely it is 
 
321
                # always wanted
 
322
                # For a key of (foo, bar, baz) create
 
323
                # _nodes_by_key[foo][bar][baz] = key_value
 
324
                for subkey in key[:-1]:
 
325
                    key_dict = key_dict.setdefault(subkey, {})
 
326
                key_dict[key[-1]] = key_value
477
327
        # cache the keys for quick set intersections
 
328
        self._keys = set(self._nodes)
478
329
        if trailers != 1:
479
330
            # there must be one line - the empty trailer line.
480
331
            raise errors.BadIndexData(self)
481
332
 
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
 
 
490
 
    def external_references(self, ref_list_num):
491
 
        """Return references that are not present in this index.
492
 
        """
493
 
        self._buffer_all()
494
 
        if ref_list_num + 1 > self.node_ref_lists:
495
 
            raise ValueError('No ref list %d, index has %d ref lists'
496
 
                % (ref_list_num, self.node_ref_lists))
497
 
        refs = set()
498
 
        nodes = self._nodes
499
 
        for key, (value, ref_lists) in nodes.iteritems():
500
 
            ref_list = ref_lists[ref_list_num]
501
 
            refs.update([ref for ref in ref_list if ref not in nodes])
502
 
        return refs
503
 
 
504
 
    def _get_nodes_by_key(self):
505
 
        if self._nodes_by_key is None:
506
 
            nodes_by_key = {}
507
 
            if self.node_ref_lists:
508
 
                for key, (value, references) in self._nodes.iteritems():
509
 
                    key_dict = nodes_by_key
510
 
                    for subkey in key[:-1]:
511
 
                        key_dict = key_dict.setdefault(subkey, {})
512
 
                    key_dict[key[-1]] = key, value, references
513
 
            else:
514
 
                for key, value in self._nodes.iteritems():
515
 
                    key_dict = nodes_by_key
516
 
                    for subkey in key[:-1]:
517
 
                        key_dict = key_dict.setdefault(subkey, {})
518
 
                    key_dict[key[-1]] = key, value
519
 
            self._nodes_by_key = nodes_by_key
520
 
        return self._nodes_by_key
521
 
 
522
333
    def iter_all_entries(self):
523
334
        """Iterate over all keys within the index.
524
335
 
568
379
 
569
380
    def _resolve_references(self, references):
570
381
        """Return the resolved key references for references.
571
 
 
 
382
        
572
383
        References are resolved by looking up the location of the key in the
573
384
        _keys_by_offset map and substituting the key name, preserving ordering.
574
385
 
575
 
        :param references: An iterable of iterables of key locations. e.g.
 
386
        :param references: An iterable of iterables of key locations. e.g. 
576
387
            [[123, 456], [123]]
577
388
        :return: A tuple of tuples of keys.
578
389
        """
632
443
 
633
444
    def _iter_entries_from_total_buffer(self, keys):
634
445
        """Iterate over keys when the entire index is parsed."""
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]
 
446
        keys = keys.intersection(self._keys)
639
447
        if self.node_ref_lists:
640
448
            for key in keys:
641
 
                value, node_refs = nodes[key]
 
449
                value, node_refs = self._nodes[key]
642
450
                yield self, key, value, node_refs
643
451
        else:
644
452
            for key in keys:
645
 
                yield self, key, nodes[key]
 
453
                yield self, key, self._nodes[key]
646
454
 
647
455
    def iter_entries(self, keys):
648
456
        """Iterate over keys within the index.
652
460
            keys supplied. No additional keys will be returned, and every
653
461
            key supplied that is in the index will be returned.
654
462
        """
 
463
        # PERFORMANCE TODO: parse and bisect all remaining data at some
 
464
        # threshold of total-index processing/get calling layers that expect to
 
465
        # read the entire index to use the iter_all_entries  method instead.
655
466
        keys = set(keys)
656
467
        if not keys:
657
468
            return []
658
469
        if self._size is None and self._nodes is None:
659
470
            self._buffer_all()
660
 
 
661
 
        # We fit about 20 keys per minimum-read (4K), so if we are looking for
662
 
        # more than 1/20th of the index its likely (assuming homogenous key
663
 
        # spread) that we'll read the entire index. If we're going to do that,
664
 
        # buffer the whole thing. A better analysis might take key spread into
665
 
        # account - but B+Tree indices are better anyway.
666
 
        # We could look at all data read, and use a threshold there, which will
667
 
        # trigger on ancestry walks, but that is not yet fully mapped out.
668
 
        if self._nodes is None and len(keys) * 20 > self.key_count():
669
 
            self._buffer_all()
670
471
        if self._nodes is not None:
671
472
            return self._iter_entries_from_total_buffer(keys)
672
473
        else:
714
515
                else:
715
516
                    yield self, key, self._nodes[key]
716
517
            return
717
 
        nodes_by_key = self._get_nodes_by_key()
718
518
        for key in keys:
719
519
            # sanity check
720
520
            if key[0] is None:
722
522
            if len(key) != self._key_length:
723
523
                raise errors.BadIndexKey(key)
724
524
            # find what it refers to:
725
 
            key_dict = nodes_by_key
 
525
            key_dict = self._nodes_by_key
726
526
            elements = list(key)
727
527
            # find the subdict whose contents should be returned.
728
528
            try:
739
539
                    # can't be empty or would not exist
740
540
                    item, value = key_dict.iteritems().next()
741
541
                    if type(value) == dict:
742
 
                        # push keys
 
542
                        # push keys 
743
543
                        dicts.extend(key_dict.itervalues())
744
544
                    else:
745
545
                        # yield keys
751
551
                # the last thing looked up was a terminal element
752
552
                yield (self, ) + key_dict
753
553
 
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
 
 
771
554
    def key_count(self):
772
555
        """Return an estimate of the number of keys in this index.
773
 
 
 
556
        
774
557
        For GraphIndex the estimate is exact.
775
558
        """
776
559
        if self._key_count is None:
792
575
        # Possible improvements:
793
576
        #  - only bisect lookup each key once
794
577
        #  - sort the keys first, and use that to reduce the bisection window
795
 
        # -----
 
578
        # ----- 
796
579
        # this progresses in three parts:
797
580
        # read data
798
581
        # parse it
807
590
                # We have the key parsed.
808
591
                continue
809
592
            index = self._parsed_key_index(key)
810
 
            if (len(self._parsed_key_map) and
 
593
            if (len(self._parsed_key_map) and 
811
594
                self._parsed_key_map[index][0] <= key and
812
595
                (self._parsed_key_map[index][1] >= key or
813
596
                 # end of the file has been parsed
817
600
                continue
818
601
            # - if we have examined this part of the file already - yes
819
602
            index = self._parsed_byte_index(location)
820
 
            if (len(self._parsed_byte_map) and
 
603
            if (len(self._parsed_byte_map) and 
821
604
                self._parsed_byte_map[index][0] <= location and
822
605
                self._parsed_byte_map[index][1] > location):
823
606
                # the byte region has been parsed, so no read is needed.
832
615
        if self._bisect_nodes is None:
833
616
            readv_ranges.append(_HEADER_READV)
834
617
        self._read_and_parse(readv_ranges)
835
 
        result = []
836
 
        if self._nodes is not None:
837
 
            # _read_and_parse triggered a _buffer_all because we requested the
838
 
            # whole data range
839
 
            for location, key in location_keys:
840
 
                if key not in self._nodes: # not present
841
 
                    result.append(((location, key), False))
842
 
                elif self.node_ref_lists:
843
 
                    value, refs = self._nodes[key]
844
 
                    result.append(((location, key),
845
 
                        (self, key, value, refs)))
846
 
                else:
847
 
                    result.append(((location, key),
848
 
                        (self, key, self._nodes[key])))
849
 
            return result
850
618
        # generate results:
851
619
        #  - figure out <, >, missing, present
852
620
        #  - result present references so we can return them.
 
621
        result = []
853
622
        # keys that we cannot answer until we resolve references
854
623
        pending_references = []
855
624
        pending_locations = set()
905
674
            if length > 0:
906
675
                readv_ranges.append((location, length))
907
676
        self._read_and_parse(readv_ranges)
908
 
        if self._nodes is not None:
909
 
            # The _read_and_parse triggered a _buffer_all, grab the data and
910
 
            # return it
911
 
            for location, key in pending_references:
912
 
                value, refs = self._nodes[key]
913
 
                result.append(((location, key), (self, key, value, refs)))
914
 
            return result
915
677
        for location, key in pending_references:
916
678
            # answer key references we had to look-up-late.
 
679
            index = self._parsed_key_index(key)
917
680
            value, refs = self._bisect_nodes[key]
918
681
            result.append(((location, key), (self, key,
919
682
                value, self._resolve_references(refs))))
1063
826
                trim_start = data.find('\n') + 1
1064
827
            else:
1065
828
                trim_start = data.find('\n', trim_start) + 1
1066
 
            if not (trim_start != 0):
1067
 
                raise AssertionError('no \n was present')
 
829
            assert trim_start != 0, 'no \n was present'
1068
830
            # print 'removing start', offset, trim_start, repr(data[:trim_start])
1069
831
        if not end_adjacent:
1070
832
            # work around python bug in rfind
1072
834
                trim_end = data.rfind('\n') + 1
1073
835
            else:
1074
836
                trim_end = data.rfind('\n', None, trim_end) + 1
1075
 
            if not (trim_end != 0):
1076
 
                raise AssertionError('no \n was present')
 
837
            assert trim_end != 0, 'no \n was present'
1077
838
            # print 'removing end', offset, trim_end, repr(data[trim_end:])
1078
839
        # adjust offset and data to the parseable data.
1079
840
        trimmed_data = data[trim_start:trim_end]
1080
 
        if not (trimmed_data):
1081
 
            raise AssertionError('read unneeded data [%d:%d] from [%d:%d]'
1082
 
                % (trim_start, trim_end, offset, offset + len(data)))
 
841
        assert trimmed_data, 'read unneeded data [%d:%d] from [%d:%d]' % (
 
842
            trim_start, trim_end, offset, offset + len(data))
1083
843
        if trim_start:
1084
844
            offset += trim_start
1085
845
        # print "parsing", repr(trimmed_data)
1103
863
            if line == '':
1104
864
                # must be at the end
1105
865
                if self._size:
1106
 
                    if not (self._size == pos + 1):
1107
 
                        raise AssertionError("%s %s" % (self._size, pos))
 
866
                    assert self._size == pos + 1, "%s %s" % (self._size, pos)
1108
867
                trailers += 1
1109
868
                continue
1110
869
            elements = line.split('\0')
1111
870
            if len(elements) != self._expected_elements:
1112
871
                raise errors.BadIndexData(self)
1113
 
            # keys are tuples. Each element is a string that may occur many
1114
 
            # times, so we intern them to save space. AB, RC, 200807
1115
 
            key = tuple([intern(element) for element in elements[:self._key_length]])
 
872
            # keys are tuples
 
873
            key = tuple(elements[:self._key_length])
1116
874
            if first_key is None:
1117
875
                first_key = key
1118
876
            absent, references, value = elements[-3:]
1185
943
            self._parsed_key_map.insert(index + 1, new_key)
1186
944
 
1187
945
    def _read_and_parse(self, readv_ranges):
1188
 
        """Read the ranges and parse the resulting data.
 
946
        """Read the the ranges and parse the resulting data.
1189
947
 
1190
948
        :param readv_ranges: A prepared readv range list.
1191
949
        """
1192
 
        if not readv_ranges:
1193
 
            return
1194
 
        if self._nodes is None and self._bytes_read * 2 >= self._size:
1195
 
            # We've already read more than 50% of the file and we are about to
1196
 
            # request more data, just _buffer_all() and be done
1197
 
            self._buffer_all()
1198
 
            return
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]
1205
 
        readv_data = self._transport.readv(self._name, readv_ranges, True,
1206
 
            self._size + self._base_offset)
1207
 
        # parse
1208
 
        for offset, data in readv_data:
1209
 
            offset -= base_offset
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
1216
 
            if offset == 0 and len(data) == self._size:
1217
 
                # We read the whole range, most likely because the
1218
 
                # Transport upcast our readv ranges into one long request
1219
 
                # for enough total data to grab the whole index.
1220
 
                self._buffer_all(StringIO(data))
1221
 
                return
1222
 
            if self._bisect_nodes is None:
1223
 
                # this must be the start
1224
 
                if not (offset == 0):
1225
 
                    raise AssertionError()
1226
 
                offset, data = self._parse_header_from_bytes(data)
1227
 
            # print readv_ranges, "[%d:%d]" % (offset, offset + len(data))
1228
 
            self._parse_region(offset, data)
 
950
        if readv_ranges:
 
951
            readv_data = self._transport.readv(self._name, readv_ranges, True,
 
952
                self._size)
 
953
            # parse
 
954
            for offset, data in readv_data:
 
955
                if self._bisect_nodes is None:
 
956
                    # this must be the start
 
957
                    assert offset == 0
 
958
                    offset, data = self._parse_header_from_bytes(data)
 
959
                # print readv_ranges, "[%d:%d]" % (offset, offset + len(data))
 
960
                self._parse_region(offset, data)
1229
961
 
1230
962
    def _signature(self):
1231
963
        """The file signature for this index type."""
1240
972
 
1241
973
class CombinedGraphIndex(object):
1242
974
    """A GraphIndex made up from smaller GraphIndices.
1243
 
 
 
975
    
1244
976
    The backing indices must implement GraphIndex, and are presumed to be
1245
977
    static data.
1246
978
 
1247
979
    Queries against the combined index will be made against the first index,
1248
 
    and then the second and so on. The order of indices can thus influence
 
980
    and then the second and so on. The order of index's can thus influence
1249
981
    performance significantly. For example, if one index is on local disk and a
1250
982
    second on a remote server, the local disk index should be before the other
1251
983
    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).
1257
984
    """
1258
985
 
1259
 
    def __init__(self, indices, reload_func=None):
 
986
    def __init__(self, indices):
1260
987
        """Create a CombinedGraphIndex backed by indices.
1261
988
 
1262
989
        :param indices: An ordered list of indices to query for data.
1263
 
        :param reload_func: A function to call if we find we are missing an
1264
 
            index. Should have the form reload_func() => True/False to indicate
1265
 
            if reloading actually changed anything.
1266
990
        """
1267
991
        self._indices = indices
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)
1276
992
 
1277
993
    def __repr__(self):
1278
994
        return "%s(%s)" % (
1279
995
                self.__class__.__name__,
1280
996
                ', '.join(map(repr, self._indices)))
1281
997
 
1282
 
    def clear_cache(self):
1283
 
        """See GraphIndex.clear_cache()"""
1284
 
        for index in self._indices:
1285
 
            index.clear_cache()
 
998
    def get_parents(self, revision_ids):
 
999
        """See StackedParentsProvider.get_parents.
 
1000
        
 
1001
        This implementation thunks the graph.Graph.get_parents api across to
 
1002
        GraphIndex.
1286
1003
 
1287
 
    def get_parent_map(self, keys):
1288
 
        """See graph.StackedParentsProvider.get_parent_map"""
1289
 
        search_keys = set(keys)
1290
 
        if NULL_REVISION in search_keys:
1291
 
            search_keys.discard(NULL_REVISION)
1292
 
            found_parents = {NULL_REVISION:[]}
1293
 
        else:
1294
 
            found_parents = {}
 
1004
        :param revision_ids: An iterable of graph keys for this graph.
 
1005
        :return: A list of parent details for each key in revision_ids.
 
1006
            Each parent details will be one of:
 
1007
             * None when the key was missing
 
1008
             * (NULL_REVISION,) when the key has no parents.
 
1009
             * (parent_key, parent_key...) otherwise.
 
1010
        """
 
1011
        search_keys = set(revision_ids)
 
1012
        search_keys.discard(NULL_REVISION)
 
1013
        found_parents = {NULL_REVISION:[]}
1295
1014
        for index, key, value, refs in self.iter_entries(search_keys):
1296
1015
            parents = refs[0]
1297
1016
            if not parents:
1298
1017
                parents = (NULL_REVISION,)
1299
1018
            found_parents[key] = parents
1300
 
        return found_parents
1301
 
 
1302
 
    has_key = _has_key_from_parent_map
1303
 
 
1304
 
    def insert_index(self, pos, index, name=None):
 
1019
        result = []
 
1020
        for key in revision_ids:
 
1021
            try:
 
1022
                result.append(found_parents[key])
 
1023
            except KeyError:
 
1024
                result.append(None)
 
1025
        return result
 
1026
 
 
1027
    def insert_index(self, pos, index):
1305
1028
        """Insert a new index in the list of indices to query.
1306
1029
 
1307
1030
        :param pos: The position to insert the index.
1308
1031
        :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)
1312
1032
        """
1313
1033
        self._indices.insert(pos, index)
1314
 
        self._index_names.insert(pos, name)
1315
1034
 
1316
1035
    def iter_all_entries(self):
1317
1036
        """Iterate over all keys within the index
1324
1043
            the most efficient order for the index.
1325
1044
        """
1326
1045
        seen_keys = set()
1327
 
        while True:
1328
 
            try:
1329
 
                for index in self._indices:
1330
 
                    for node in index.iter_all_entries():
1331
 
                        if node[1] not in seen_keys:
1332
 
                            yield node
1333
 
                            seen_keys.add(node[1])
1334
 
                return
1335
 
            except errors.NoSuchFile:
1336
 
                self._reload_or_raise()
 
1046
        for index in self._indices:
 
1047
            for node in index.iter_all_entries():
 
1048
                if node[1] not in seen_keys:
 
1049
                    yield node
 
1050
                    seen_keys.add(node[1])
1337
1051
 
1338
1052
    def iter_entries(self, keys):
1339
1053
        """Iterate over keys within the index.
1342
1056
        value and are only reported once.
1343
1057
 
1344
1058
        :param keys: An iterable providing the keys to be retrieved.
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
 
1059
        :return: An iterable of (index, key, reference_lists, value). There is no
 
1060
            defined order for the result iteration - it will be in the most
1347
1061
            efficient order for the index.
1348
1062
        """
1349
1063
        keys = set(keys)
1350
 
        hit_indices = []
1351
 
        while True:
1352
 
            try:
1353
 
                for index in self._indices:
1354
 
                    if not keys:
1355
 
                        break
1356
 
                    index_hit = False
1357
 
                    for node in index.iter_entries(keys):
1358
 
                        keys.remove(node[1])
1359
 
                        yield node
1360
 
                        index_hit = True
1361
 
                    if index_hit:
1362
 
                        hit_indices.append(index)
1363
 
                break
1364
 
            except errors.NoSuchFile:
1365
 
                self._reload_or_raise()
1366
 
        self._move_to_front(hit_indices)
 
1064
        for index in self._indices:
 
1065
            if not keys:
 
1066
                return
 
1067
            for node in index.iter_entries(keys):
 
1068
                keys.remove(node[1])
 
1069
                yield node
1367
1070
 
1368
1071
    def iter_entries_prefix(self, keys):
1369
1072
        """Iterate over keys within the index using prefix matching.
1389
1092
        if not keys:
1390
1093
            return
1391
1094
        seen_keys = set()
1392
 
        hit_indices = []
1393
 
        while True:
1394
 
            try:
1395
 
                for index in self._indices:
1396
 
                    index_hit = False
1397
 
                    for node in index.iter_entries_prefix(keys):
1398
 
                        if node[1] in seen_keys:
1399
 
                            continue
1400
 
                        seen_keys.add(node[1])
1401
 
                        yield node
1402
 
                        index_hit = True
1403
 
                    if index_hit:
1404
 
                        hit_indices.append(index)
1405
 
                break
1406
 
            except errors.NoSuchFile:
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
 
1095
        for index in self._indices:
 
1096
            for node in index.iter_entries_prefix(keys):
 
1097
                if node[1] in seen_keys:
 
1098
                    continue
 
1099
                seen_keys.add(node[1])
 
1100
                yield node
1539
1101
 
1540
1102
    def key_count(self):
1541
1103
        """Return an estimate of the number of keys in this index.
1542
 
 
 
1104
        
1543
1105
        For CombinedGraphIndex this is approximated by the sum of the keys of
1544
1106
        the child indices. As child indices may have duplicate keys this can
1545
1107
        have a maximum error of the number of child indices * largest number of
1546
1108
        keys in any index.
1547
1109
        """
1548
 
        while True:
1549
 
            try:
1550
 
                return sum((index.key_count() for index in self._indices), 0)
1551
 
            except errors.NoSuchFile:
1552
 
                self._reload_or_raise()
1553
 
 
1554
 
    missing_keys = _missing_keys_from_parent_map
1555
 
 
1556
 
    def _reload_or_raise(self):
1557
 
        """We just got a NoSuchFile exception.
1558
 
 
1559
 
        Try to reload the indices, if it fails, just raise the current
1560
 
        exception.
1561
 
        """
1562
 
        if self._reload_func is None:
1563
 
            raise
1564
 
        exc_type, exc_value, exc_traceback = sys.exc_info()
1565
 
        trace.mutter('Trying to reload after getting exception: %s',
1566
 
                     exc_value)
1567
 
        if not self._reload_func():
1568
 
            # We tried to reload, but nothing changed, so we fail anyway
1569
 
            trace.mutter('_reload_func indicated nothing has changed.'
1570
 
                         ' Raising original exception.')
1571
 
            raise exc_type, exc_value, exc_traceback
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
 
1110
        return sum((index.key_count() for index in self._indices), 0)
1577
1111
 
1578
1112
    def validate(self):
1579
1113
        """Validate that everything in the index can be accessed."""
1580
 
        while True:
1581
 
            try:
1582
 
                for index in self._indices:
1583
 
                    index.validate()
1584
 
                return
1585
 
            except errors.NoSuchFile:
1586
 
                self._reload_or_raise()
 
1114
        for index in self._indices:
 
1115
            index.validate()
1587
1116
 
1588
1117
 
1589
1118
class InMemoryGraphIndex(GraphIndexBuilder):
1633
1162
            defined order for the result iteration - it will be in the most
1634
1163
            efficient order for the index (keys iteration order in this case).
1635
1164
        """
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]
 
1165
        keys = set(keys)
1640
1166
        if self.reference_lists:
1641
 
            for key in keys:
1642
 
                node = nodes[key]
 
1167
            for key in keys.intersection(self._keys):
 
1168
                node = self._nodes[key]
1643
1169
                if not node[0]:
1644
1170
                    yield self, key, node[2], node[1]
1645
1171
        else:
1646
 
            for key in keys:
1647
 
                node = nodes[key]
 
1172
            for key in keys.intersection(self._keys):
 
1173
                node = self._nodes[key]
1648
1174
                if not node[0]:
1649
1175
                    yield self, key, node[2]
1650
1176
 
1679
1205
                    raise errors.BadIndexKey(key)
1680
1206
                node = self._nodes[key]
1681
1207
                if node[0]:
1682
 
                    continue
 
1208
                    continue 
1683
1209
                if self.reference_lists:
1684
1210
                    yield self, key, node[2], node[1]
1685
1211
                else:
1686
1212
                    yield self, key, node[2]
1687
1213
            return
1688
 
        nodes_by_key = self._get_nodes_by_key()
1689
1214
        for key in keys:
1690
1215
            # sanity check
1691
1216
            if key[0] is None:
1693
1218
            if len(key) != self._key_length:
1694
1219
                raise errors.BadIndexKey(key)
1695
1220
            # find what it refers to:
1696
 
            key_dict = nodes_by_key
 
1221
            key_dict = self._nodes_by_key
1697
1222
            elements = list(key)
1698
1223
            # find the subdict to return
1699
1224
            try:
1710
1235
                    # can't be empty or would not exist
1711
1236
                    item, value = key_dict.iteritems().next()
1712
1237
                    if type(value) == dict:
1713
 
                        # push keys
 
1238
                        # push keys 
1714
1239
                        dicts.extend(key_dict.itervalues())
1715
1240
                    else:
1716
1241
                        # yield keys
1721
1246
 
1722
1247
    def key_count(self):
1723
1248
        """Return an estimate of the number of keys in this index.
1724
 
 
 
1249
        
1725
1250
        For InMemoryGraphIndex the estimate is exact.
1726
1251
        """
1727
 
        return len(self._nodes) - len(self._absent_keys)
 
1252
        return len(self._keys)
1728
1253
 
1729
1254
    def validate(self):
1730
1255
        """In memory index's have no known corruption at the moment."""
1735
1260
 
1736
1261
    Queries against this will emit queries against the adapted Graph with the
1737
1262
    prefix added, queries for all items use iter_entries_prefix. The returned
1738
 
    nodes will have their keys and node references adjusted to remove the
 
1263
    nodes will have their keys and node references adjusted to remove the 
1739
1264
    prefix. Finally, an add_nodes_callback can be supplied - when called the
1740
1265
    nodes and references being added will have prefix prepended.
1741
1266
    """
1768
1293
                    adjusted_references))
1769
1294
        except ValueError:
1770
1295
            # XXX: TODO add an explicit interface for getting the reference list
1771
 
            # status, to handle this bit of user-friendliness in the API more
 
1296
            # status, to handle this bit of user-friendliness in the API more 
1772
1297
            # explicitly.
1773
1298
            for (key, value) in nodes:
1774
1299
                translated_nodes.append((self.prefix + key, value))
1846
1371
 
1847
1372
    def key_count(self):
1848
1373
        """Return an estimate of the number of keys in this index.
1849
 
 
 
1374
        
1850
1375
        For GraphIndexPrefixAdapter this is relatively expensive - key
1851
1376
        iteration with the prefix is done.
1852
1377
        """