~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/index.py

  • Committer: Jelmer Vernooij
  • Date: 2008-06-11 18:58:19 UTC
  • mto: (3649.3.2 bzr.dev)
  • mto: This revision was merged to the branch mainline in revision 3658.
  • Revision ID: jelmer@samba.org-20080611185819-o4shi1ranh9zh01e
Move ftp transport into separate directory.

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
96
 
        # A dict of {key: (absent, ref_lists, value)}
 
82
        self._keys = set()
97
83
        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
 
84
        self._nodes_by_key = {}
101
85
        self._key_length = key_elements
102
 
        self._optimize_for_size = False
103
 
        self._combine_backing_indices = True
104
86
 
105
87
    def _check_key(self, key):
106
88
        """Raise BadIndexKey if key is not a valid key for this index."""
107
 
        if type(key) not in (tuple, StaticTuple):
 
89
        if type(key) != tuple:
108
90
            raise errors.BadIndexKey(key)
109
91
        if self._key_length != len(key):
110
92
            raise errors.BadIndexKey(key)
112
94
            if not element or _whitespace_re.search(element) is not None:
113
95
                raise errors.BadIndexKey(element)
114
96
 
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
 
97
    def add_node(self, key, value, references=()):
 
98
        """Add a node to the index.
 
99
 
 
100
        :param key: The key. keys are non-empty tuples containing
 
101
            as many whitespace-free utf8 bytestrings as the key length
 
102
            defined for this index.
 
103
        :param references: An iterable of iterables of keys. Each is a
 
104
            reference to another key.
 
105
        :param value: The value to associate with the key. It may be any
 
106
            bytes as long as it does not contain \0 or \n.
 
107
        """
194
108
        self._check_key(key)
195
109
        if _newline_null_re.search(value) is not None:
196
110
            raise errors.BadIndexValue(value)
197
111
        if len(references) != self.reference_lists:
198
112
            raise errors.BadIndexValue(references)
199
113
        node_refs = []
200
 
        absent_references = []
201
114
        for reference_list in references:
202
115
            for reference in reference_list:
203
 
                # If reference *is* in self._nodes, then we know it has already
204
 
                # been checked.
 
116
                self._check_key(reference)
205
117
                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':
 
118
                    self._nodes[reference] = ('a', (), '')
 
119
            node_refs.append(tuple(reference_list))
 
120
        if key in self._nodes and self._nodes[key][0] == '':
227
121
            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
 
        
 
122
        self._nodes[key] = ('', tuple(node_refs), value)
 
123
        self._keys.add(key)
 
124
        if self._key_length > 1:
 
125
            key_dict = self._nodes_by_key
 
126
            if self.reference_lists:
 
127
                key_value = key, value, tuple(node_refs)
 
128
            else:
 
129
                key_value = key, value
 
130
            # possibly should do this on-demand, but it seems likely it is 
 
131
            # always wanted
 
132
            # For a key of (foo, bar, baz) create
 
133
            # _nodes_by_key[foo][bar][baz] = key_value
 
134
            for subkey in key[:-1]:
 
135
                key_dict = key_dict.setdefault(subkey, {})
 
136
            key_dict[key[-1]] = key_value
 
137
 
245
138
    def finish(self):
246
139
        lines = [_SIGNATURE]
247
140
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
248
141
        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')
 
142
        lines.append(_OPTION_LEN + str(len(self._keys)) + '\n')
251
143
        prefix_length = sum(len(x) for x in lines)
252
144
        # references are byte offsets. To avoid having to do nasty
253
 
        # polynomial work to resolve offsets (references to later in the
 
145
        # polynomial work to resolve offsets (references to later in the 
254
146
        # file cannot be determined until all the inbetween references have
255
147
        # been calculated too) we pad the offsets with 0's to make them be
256
148
        # of consistent length. Using binary offsets would break the trivial
260
152
        # one to pad all the data with reference-length and determine entry
261
153
        # addresses.
262
154
        # One to serialise.
263
 
 
 
155
        
264
156
        # forward sorted by key. In future we may consider topological sorting,
265
157
        # at the cost of table scans for direct lookup, or a second index for
266
158
        # direct lookup
327
219
            raise errors.BzrError('Failed index creation. Internal error:'
328
220
                ' mismatched output length and expected length: %d %d' %
329
221
                (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
 
222
        return StringIO(''.join(lines))
365
223
 
366
224
 
367
225
class GraphIndex(object):
368
226
    """An index for data with embedded graphs.
369
 
 
 
227
 
370
228
    The index maps keys to a list of key reference lists, and a value.
371
229
    Each node has the same number of key reference lists. Each key reference
372
230
    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
 
231
    terminated string without any newlines. The storage of the index is 
374
232
    hidden in the interface: keys and key references are always tuples of
375
233
    bytestrings, never the internal representation (e.g. dictionary offsets).
376
234
 
382
240
    suitable for production use. :XXX
383
241
    """
384
242
 
385
 
    def __init__(self, transport, name, size, unlimited_cache=False, offset=0):
 
243
    def __init__(self, transport, name, size):
386
244
        """Open an index called name on transport.
387
245
 
388
246
        :param transport: A bzrlib.transport.Transport.
394
252
            avoided by having it supplied. If size is None, then bisection
395
253
            support will be disabled and accessing the index will just stream
396
254
            all the data.
397
 
        :param offset: Instead of starting the index data at offset 0, start it
398
 
            at an arbitrary offset.
399
255
        """
400
256
        self._transport = transport
401
257
        self._name = name
416
272
        self._keys_by_offset = None
417
273
        self._nodes_by_key = None
418
274
        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
275
 
423
276
    def __eq__(self, other):
424
277
        """Equal when self and other were created with the same parameters."""
431
284
    def __ne__(self, other):
432
285
        return not self.__eq__(other)
433
286
 
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):
 
287
    def _buffer_all(self):
439
288
        """Buffer all the index data.
440
289
 
441
290
        Mutates self._nodes and self.keys_by_offset.
442
291
        """
443
 
        if self._nodes is not None:
444
 
            # We already did this
445
 
            return
446
292
        if 'index' in debug.debug_flags:
447
293
            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:])
 
294
        stream = self._transport.get(self._name)
454
295
        self._read_prefix(stream)
455
296
        self._expected_elements = 3 + self._key_length
456
297
        line_count = 0
458
299
        self._keys_by_offset = {}
459
300
        # ready-to-return key:value or key:value, node_ref_lists
460
301
        self._nodes = {}
461
 
        self._nodes_by_key = None
 
302
        self._nodes_by_key = {}
462
303
        trailers = 0
463
304
        pos = stream.tell()
464
305
        lines = stream.read().split('\n')
465
 
        # GZ 2009-09-20: Should really use a try/finally block to ensure close
466
 
        stream.close()
467
306
        del lines[-1]
468
307
        _, _, _, trailers = self._parse_lines(lines, pos)
469
308
        for key, absent, references, value in self._keys_by_offset.itervalues():
475
314
            else:
476
315
                node_value = value
477
316
            self._nodes[key] = node_value
 
317
            if self._key_length > 1:
 
318
                subkey = list(reversed(key[:-1]))
 
319
                key_dict = self._nodes_by_key
 
320
                if self.node_ref_lists:
 
321
                    key_value = key, node_value[0], node_value[1]
 
322
                else:
 
323
                    key_value = key, node_value
 
324
                # possibly should do this on-demand, but it seems likely it is 
 
325
                # always wanted
 
326
                # For a key of (foo, bar, baz) create
 
327
                # _nodes_by_key[foo][bar][baz] = key_value
 
328
                for subkey in key[:-1]:
 
329
                    key_dict = key_dict.setdefault(subkey, {})
 
330
                key_dict[key[-1]] = key_value
478
331
        # cache the keys for quick set intersections
 
332
        self._keys = set(self._nodes)
479
333
        if trailers != 1:
480
334
            # there must be one line - the empty trailer line.
481
335
            raise errors.BadIndexData(self)
482
336
 
483
 
    def clear_cache(self):
484
 
        """Clear out any cached/memoized values.
485
 
 
486
 
        This can be called at any time, but generally it is used when we have
487
 
        extracted some information, but don't expect to be requesting any more
488
 
        from this index.
489
 
        """
490
 
 
491
 
    def external_references(self, ref_list_num):
492
 
        """Return references that are not present in this index.
493
 
        """
494
 
        self._buffer_all()
495
 
        if ref_list_num + 1 > self.node_ref_lists:
496
 
            raise ValueError('No ref list %d, index has %d ref lists'
497
 
                % (ref_list_num, self.node_ref_lists))
498
 
        refs = set()
499
 
        nodes = self._nodes
500
 
        for key, (value, ref_lists) in nodes.iteritems():
501
 
            ref_list = ref_lists[ref_list_num]
502
 
            refs.update([ref for ref in ref_list if ref not in nodes])
503
 
        return refs
504
 
 
505
 
    def _get_nodes_by_key(self):
506
 
        if self._nodes_by_key is None:
507
 
            nodes_by_key = {}
508
 
            if self.node_ref_lists:
509
 
                for key, (value, references) in self._nodes.iteritems():
510
 
                    key_dict = nodes_by_key
511
 
                    for subkey in key[:-1]:
512
 
                        key_dict = key_dict.setdefault(subkey, {})
513
 
                    key_dict[key[-1]] = key, value, references
514
 
            else:
515
 
                for key, value in self._nodes.iteritems():
516
 
                    key_dict = nodes_by_key
517
 
                    for subkey in key[:-1]:
518
 
                        key_dict = key_dict.setdefault(subkey, {})
519
 
                    key_dict[key[-1]] = key, value
520
 
            self._nodes_by_key = nodes_by_key
521
 
        return self._nodes_by_key
522
 
 
523
337
    def iter_all_entries(self):
524
338
        """Iterate over all keys within the index.
525
339
 
569
383
 
570
384
    def _resolve_references(self, references):
571
385
        """Return the resolved key references for references.
572
 
 
 
386
        
573
387
        References are resolved by looking up the location of the key in the
574
388
        _keys_by_offset map and substituting the key name, preserving ordering.
575
389
 
576
 
        :param references: An iterable of iterables of key locations. e.g.
 
390
        :param references: An iterable of iterables of key locations. e.g. 
577
391
            [[123, 456], [123]]
578
392
        :return: A tuple of tuples of keys.
579
393
        """
633
447
 
634
448
    def _iter_entries_from_total_buffer(self, keys):
635
449
        """Iterate over keys when the entire index is parsed."""
636
 
        # Note: See the note in BTreeBuilder.iter_entries for why we don't use
637
 
        #       .intersection() here
638
 
        nodes = self._nodes
639
 
        keys = [key for key in keys if key in nodes]
 
450
        keys = keys.intersection(self._keys)
640
451
        if self.node_ref_lists:
641
452
            for key in keys:
642
 
                value, node_refs = nodes[key]
 
453
                value, node_refs = self._nodes[key]
643
454
                yield self, key, value, node_refs
644
455
        else:
645
456
            for key in keys:
646
 
                yield self, key, nodes[key]
 
457
                yield self, key, self._nodes[key]
647
458
 
648
459
    def iter_entries(self, keys):
649
460
        """Iterate over keys within the index.
653
464
            keys supplied. No additional keys will be returned, and every
654
465
            key supplied that is in the index will be returned.
655
466
        """
 
467
        # PERFORMANCE TODO: parse and bisect all remaining data at some
 
468
        # threshold of total-index processing/get calling layers that expect to
 
469
        # read the entire index to use the iter_all_entries  method instead.
656
470
        keys = set(keys)
657
471
        if not keys:
658
472
            return []
659
473
        if self._size is None and self._nodes is None:
660
474
            self._buffer_all()
661
 
 
662
 
        # We fit about 20 keys per minimum-read (4K), so if we are looking for
663
 
        # more than 1/20th of the index its likely (assuming homogenous key
664
 
        # spread) that we'll read the entire index. If we're going to do that,
665
 
        # buffer the whole thing. A better analysis might take key spread into
666
 
        # account - but B+Tree indices are better anyway.
667
 
        # We could look at all data read, and use a threshold there, which will
668
 
        # trigger on ancestry walks, but that is not yet fully mapped out.
669
 
        if self._nodes is None and len(keys) * 20 > self.key_count():
670
 
            self._buffer_all()
671
475
        if self._nodes is not None:
672
476
            return self._iter_entries_from_total_buffer(keys)
673
477
        else:
715
519
                else:
716
520
                    yield self, key, self._nodes[key]
717
521
            return
718
 
        nodes_by_key = self._get_nodes_by_key()
719
522
        for key in keys:
720
523
            # sanity check
721
524
            if key[0] is None:
723
526
            if len(key) != self._key_length:
724
527
                raise errors.BadIndexKey(key)
725
528
            # find what it refers to:
726
 
            key_dict = nodes_by_key
 
529
            key_dict = self._nodes_by_key
727
530
            elements = list(key)
728
531
            # find the subdict whose contents should be returned.
729
532
            try:
740
543
                    # can't be empty or would not exist
741
544
                    item, value = key_dict.iteritems().next()
742
545
                    if type(value) == dict:
743
 
                        # push keys
 
546
                        # push keys 
744
547
                        dicts.extend(key_dict.itervalues())
745
548
                    else:
746
549
                        # yield keys
752
555
                # the last thing looked up was a terminal element
753
556
                yield (self, ) + key_dict
754
557
 
755
 
    def _find_ancestors(self, keys, ref_list_num, parent_map, missing_keys):
756
 
        """See BTreeIndex._find_ancestors."""
757
 
        # The api can be implemented as a trivial overlay on top of
758
 
        # iter_entries, it is not an efficient implementation, but it at least
759
 
        # gets the job done.
760
 
        found_keys = set()
761
 
        search_keys = set()
762
 
        for index, key, value, refs in self.iter_entries(keys):
763
 
            parent_keys = refs[ref_list_num]
764
 
            found_keys.add(key)
765
 
            parent_map[key] = parent_keys
766
 
            search_keys.update(parent_keys)
767
 
        # Figure out what, if anything, was missing
768
 
        missing_keys.update(set(keys).difference(found_keys))
769
 
        search_keys = search_keys.difference(parent_map)
770
 
        return search_keys
771
 
 
772
558
    def key_count(self):
773
559
        """Return an estimate of the number of keys in this index.
774
 
 
 
560
        
775
561
        For GraphIndex the estimate is exact.
776
562
        """
777
563
        if self._key_count is None:
793
579
        # Possible improvements:
794
580
        #  - only bisect lookup each key once
795
581
        #  - sort the keys first, and use that to reduce the bisection window
796
 
        # -----
 
582
        # ----- 
797
583
        # this progresses in three parts:
798
584
        # read data
799
585
        # parse it
808
594
                # We have the key parsed.
809
595
                continue
810
596
            index = self._parsed_key_index(key)
811
 
            if (len(self._parsed_key_map) and
 
597
            if (len(self._parsed_key_map) and 
812
598
                self._parsed_key_map[index][0] <= key and
813
599
                (self._parsed_key_map[index][1] >= key or
814
600
                 # end of the file has been parsed
818
604
                continue
819
605
            # - if we have examined this part of the file already - yes
820
606
            index = self._parsed_byte_index(location)
821
 
            if (len(self._parsed_byte_map) and
 
607
            if (len(self._parsed_byte_map) and 
822
608
                self._parsed_byte_map[index][0] <= location and
823
609
                self._parsed_byte_map[index][1] > location):
824
610
                # the byte region has been parsed, so no read is needed.
833
619
        if self._bisect_nodes is None:
834
620
            readv_ranges.append(_HEADER_READV)
835
621
        self._read_and_parse(readv_ranges)
836
 
        result = []
837
 
        if self._nodes is not None:
838
 
            # _read_and_parse triggered a _buffer_all because we requested the
839
 
            # whole data range
840
 
            for location, key in location_keys:
841
 
                if key not in self._nodes: # not present
842
 
                    result.append(((location, key), False))
843
 
                elif self.node_ref_lists:
844
 
                    value, refs = self._nodes[key]
845
 
                    result.append(((location, key),
846
 
                        (self, key, value, refs)))
847
 
                else:
848
 
                    result.append(((location, key),
849
 
                        (self, key, self._nodes[key])))
850
 
            return result
851
622
        # generate results:
852
623
        #  - figure out <, >, missing, present
853
624
        #  - result present references so we can return them.
 
625
        result = []
854
626
        # keys that we cannot answer until we resolve references
855
627
        pending_references = []
856
628
        pending_locations = set()
906
678
            if length > 0:
907
679
                readv_ranges.append((location, length))
908
680
        self._read_and_parse(readv_ranges)
909
 
        if self._nodes is not None:
910
 
            # The _read_and_parse triggered a _buffer_all, grab the data and
911
 
            # return it
912
 
            for location, key in pending_references:
913
 
                value, refs = self._nodes[key]
914
 
                result.append(((location, key), (self, key, value, refs)))
915
 
            return result
916
681
        for location, key in pending_references:
917
682
            # answer key references we had to look-up-late.
 
683
            index = self._parsed_key_index(key)
918
684
            value, refs = self._bisect_nodes[key]
919
685
            result.append(((location, key), (self, key,
920
686
                value, self._resolve_references(refs))))
1064
830
                trim_start = data.find('\n') + 1
1065
831
            else:
1066
832
                trim_start = data.find('\n', trim_start) + 1
1067
 
            if not (trim_start != 0):
1068
 
                raise AssertionError('no \n was present')
 
833
            assert trim_start != 0, 'no \n was present'
1069
834
            # print 'removing start', offset, trim_start, repr(data[:trim_start])
1070
835
        if not end_adjacent:
1071
836
            # work around python bug in rfind
1073
838
                trim_end = data.rfind('\n') + 1
1074
839
            else:
1075
840
                trim_end = data.rfind('\n', None, trim_end) + 1
1076
 
            if not (trim_end != 0):
1077
 
                raise AssertionError('no \n was present')
 
841
            assert trim_end != 0, 'no \n was present'
1078
842
            # print 'removing end', offset, trim_end, repr(data[trim_end:])
1079
843
        # adjust offset and data to the parseable data.
1080
844
        trimmed_data = data[trim_start:trim_end]
1081
 
        if not (trimmed_data):
1082
 
            raise AssertionError('read unneeded data [%d:%d] from [%d:%d]'
1083
 
                % (trim_start, trim_end, offset, offset + len(data)))
 
845
        assert trimmed_data, 'read unneeded data [%d:%d] from [%d:%d]' % (
 
846
            trim_start, trim_end, offset, offset + len(data))
1084
847
        if trim_start:
1085
848
            offset += trim_start
1086
849
        # print "parsing", repr(trimmed_data)
1104
867
            if line == '':
1105
868
                # must be at the end
1106
869
                if self._size:
1107
 
                    if not (self._size == pos + 1):
1108
 
                        raise AssertionError("%s %s" % (self._size, pos))
 
870
                    assert self._size == pos + 1, "%s %s" % (self._size, pos)
1109
871
                trailers += 1
1110
872
                continue
1111
873
            elements = line.split('\0')
1112
874
            if len(elements) != self._expected_elements:
1113
875
                raise errors.BadIndexData(self)
1114
 
            # keys are tuples. Each element is a string that may occur many
1115
 
            # times, so we intern them to save space. AB, RC, 200807
1116
 
            key = tuple([intern(element) for element in elements[:self._key_length]])
 
876
            # keys are tuples
 
877
            key = tuple(elements[:self._key_length])
1117
878
            if first_key is None:
1118
879
                first_key = key
1119
880
            absent, references, value = elements[-3:]
1186
947
            self._parsed_key_map.insert(index + 1, new_key)
1187
948
 
1188
949
    def _read_and_parse(self, readv_ranges):
1189
 
        """Read the ranges and parse the resulting data.
 
950
        """Read the the ranges and parse the resulting data.
1190
951
 
1191
952
        :param readv_ranges: A prepared readv range list.
1192
953
        """
1193
 
        if not readv_ranges:
1194
 
            return
1195
 
        if self._nodes is None and self._bytes_read * 2 >= self._size:
1196
 
            # We've already read more than 50% of the file and we are about to
1197
 
            # request more data, just _buffer_all() and be done
1198
 
            self._buffer_all()
1199
 
            return
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]
1206
 
        readv_data = self._transport.readv(self._name, readv_ranges, True,
1207
 
            self._size + self._base_offset)
1208
 
        # parse
1209
 
        for offset, data in readv_data:
1210
 
            offset -= base_offset
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
1217
 
            if offset == 0 and len(data) == self._size:
1218
 
                # We read the whole range, most likely because the
1219
 
                # Transport upcast our readv ranges into one long request
1220
 
                # for enough total data to grab the whole index.
1221
 
                self._buffer_all(StringIO(data))
1222
 
                return
1223
 
            if self._bisect_nodes is None:
1224
 
                # this must be the start
1225
 
                if not (offset == 0):
1226
 
                    raise AssertionError()
1227
 
                offset, data = self._parse_header_from_bytes(data)
1228
 
            # print readv_ranges, "[%d:%d]" % (offset, offset + len(data))
1229
 
            self._parse_region(offset, data)
 
954
        if readv_ranges:
 
955
            readv_data = self._transport.readv(self._name, readv_ranges, True,
 
956
                self._size)
 
957
            # parse
 
958
            for offset, data in readv_data:
 
959
                if self._bisect_nodes is None:
 
960
                    # this must be the start
 
961
                    assert offset == 0
 
962
                    offset, data = self._parse_header_from_bytes(data)
 
963
                # print readv_ranges, "[%d:%d]" % (offset, offset + len(data))
 
964
                self._parse_region(offset, data)
1230
965
 
1231
966
    def _signature(self):
1232
967
        """The file signature for this index type."""
1241
976
 
1242
977
class CombinedGraphIndex(object):
1243
978
    """A GraphIndex made up from smaller GraphIndices.
1244
 
 
 
979
    
1245
980
    The backing indices must implement GraphIndex, and are presumed to be
1246
981
    static data.
1247
982
 
1248
983
    Queries against the combined index will be made against the first index,
1249
 
    and then the second and so on. The order of indices can thus influence
 
984
    and then the second and so on. The order of index's can thus influence
1250
985
    performance significantly. For example, if one index is on local disk and a
1251
986
    second on a remote server, the local disk index should be before the other
1252
987
    in the index list.
1253
 
    
1254
 
    Also, queries tend to need results from the same indices as previous
1255
 
    queries.  So the indices will be reordered after every query to put the
1256
 
    indices that had the result(s) of that query first (while otherwise
1257
 
    preserving the relative ordering).
1258
988
    """
1259
989
 
1260
 
    def __init__(self, indices, reload_func=None):
 
990
    def __init__(self, indices):
1261
991
        """Create a CombinedGraphIndex backed by indices.
1262
992
 
1263
993
        :param indices: An ordered list of indices to query for data.
1264
 
        :param reload_func: A function to call if we find we are missing an
1265
 
            index. Should have the form reload_func() => True/False to indicate
1266
 
            if reloading actually changed anything.
1267
994
        """
1268
995
        self._indices = indices
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)
1277
996
 
1278
997
    def __repr__(self):
1279
998
        return "%s(%s)" % (
1280
999
                self.__class__.__name__,
1281
1000
                ', '.join(map(repr, self._indices)))
1282
1001
 
1283
 
    def clear_cache(self):
1284
 
        """See GraphIndex.clear_cache()"""
1285
 
        for index in self._indices:
1286
 
            index.clear_cache()
 
1002
    @symbol_versioning.deprecated_method(symbol_versioning.one_one)
 
1003
    def get_parents(self, revision_ids):
 
1004
        """See graph._StackedParentsProvider.get_parents.
 
1005
        
 
1006
        This implementation thunks the graph.Graph.get_parents api across to
 
1007
        GraphIndex.
 
1008
 
 
1009
        :param revision_ids: An iterable of graph keys for this graph.
 
1010
        :return: A list of parent details for each key in revision_ids.
 
1011
            Each parent details will be one of:
 
1012
             * None when the key was missing
 
1013
             * (NULL_REVISION,) when the key has no parents.
 
1014
             * (parent_key, parent_key...) otherwise.
 
1015
        """
 
1016
        parent_map = self.get_parent_map(revision_ids)
 
1017
        return [parent_map.get(r, None) for r in revision_ids]
1287
1018
 
1288
1019
    def get_parent_map(self, keys):
1289
 
        """See graph.StackedParentsProvider.get_parent_map"""
 
1020
        """See graph._StackedParentsProvider.get_parent_map"""
1290
1021
        search_keys = set(keys)
1291
1022
        if NULL_REVISION in search_keys:
1292
1023
            search_keys.discard(NULL_REVISION)
1300
1031
            found_parents[key] = parents
1301
1032
        return found_parents
1302
1033
 
1303
 
    has_key = _has_key_from_parent_map
1304
 
 
1305
 
    def insert_index(self, pos, index, name=None):
 
1034
    def insert_index(self, pos, index):
1306
1035
        """Insert a new index in the list of indices to query.
1307
1036
 
1308
1037
        :param pos: The position to insert the index.
1309
1038
        :param index: The index to insert.
1310
 
        :param name: a name for this index, e.g. a pack name.  These names can
1311
 
            be used to reflect index reorderings to related CombinedGraphIndex
1312
 
            instances that use the same names.  (see set_sibling_indices)
1313
1039
        """
1314
1040
        self._indices.insert(pos, index)
1315
 
        self._index_names.insert(pos, name)
1316
1041
 
1317
1042
    def iter_all_entries(self):
1318
1043
        """Iterate over all keys within the index
1325
1050
            the most efficient order for the index.
1326
1051
        """
1327
1052
        seen_keys = set()
1328
 
        while True:
1329
 
            try:
1330
 
                for index in self._indices:
1331
 
                    for node in index.iter_all_entries():
1332
 
                        if node[1] not in seen_keys:
1333
 
                            yield node
1334
 
                            seen_keys.add(node[1])
1335
 
                return
1336
 
            except errors.NoSuchFile:
1337
 
                self._reload_or_raise()
 
1053
        for index in self._indices:
 
1054
            for node in index.iter_all_entries():
 
1055
                if node[1] not in seen_keys:
 
1056
                    yield node
 
1057
                    seen_keys.add(node[1])
1338
1058
 
1339
1059
    def iter_entries(self, keys):
1340
1060
        """Iterate over keys within the index.
1343
1063
        value and are only reported once.
1344
1064
 
1345
1065
        :param keys: An iterable providing the keys to be retrieved.
1346
 
        :return: An iterable of (index, key, reference_lists, value). There is
1347
 
            no defined order for the result iteration - it will be in the most
 
1066
        :return: An iterable of (index, key, reference_lists, value). There is no
 
1067
            defined order for the result iteration - it will be in the most
1348
1068
            efficient order for the index.
1349
1069
        """
1350
1070
        keys = set(keys)
1351
 
        hit_indices = []
1352
 
        while True:
1353
 
            try:
1354
 
                for index in self._indices:
1355
 
                    if not keys:
1356
 
                        break
1357
 
                    index_hit = False
1358
 
                    for node in index.iter_entries(keys):
1359
 
                        keys.remove(node[1])
1360
 
                        yield node
1361
 
                        index_hit = True
1362
 
                    if index_hit:
1363
 
                        hit_indices.append(index)
1364
 
                break
1365
 
            except errors.NoSuchFile:
1366
 
                self._reload_or_raise()
1367
 
        self._move_to_front(hit_indices)
 
1071
        for index in self._indices:
 
1072
            if not keys:
 
1073
                return
 
1074
            for node in index.iter_entries(keys):
 
1075
                keys.remove(node[1])
 
1076
                yield node
1368
1077
 
1369
1078
    def iter_entries_prefix(self, keys):
1370
1079
        """Iterate over keys within the index using prefix matching.
1390
1099
        if not keys:
1391
1100
            return
1392
1101
        seen_keys = set()
1393
 
        hit_indices = []
1394
 
        while True:
1395
 
            try:
1396
 
                for index in self._indices:
1397
 
                    index_hit = False
1398
 
                    for node in index.iter_entries_prefix(keys):
1399
 
                        if node[1] in seen_keys:
1400
 
                            continue
1401
 
                        seen_keys.add(node[1])
1402
 
                        yield node
1403
 
                        index_hit = True
1404
 
                    if index_hit:
1405
 
                        hit_indices.append(index)
1406
 
                break
1407
 
            except errors.NoSuchFile:
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
 
1102
        for index in self._indices:
 
1103
            for node in index.iter_entries_prefix(keys):
 
1104
                if node[1] in seen_keys:
 
1105
                    continue
 
1106
                seen_keys.add(node[1])
 
1107
                yield node
1540
1108
 
1541
1109
    def key_count(self):
1542
1110
        """Return an estimate of the number of keys in this index.
1543
 
 
 
1111
        
1544
1112
        For CombinedGraphIndex this is approximated by the sum of the keys of
1545
1113
        the child indices. As child indices may have duplicate keys this can
1546
1114
        have a maximum error of the number of child indices * largest number of
1547
1115
        keys in any index.
1548
1116
        """
1549
 
        while True:
1550
 
            try:
1551
 
                return sum((index.key_count() for index in self._indices), 0)
1552
 
            except errors.NoSuchFile:
1553
 
                self._reload_or_raise()
1554
 
 
1555
 
    missing_keys = _missing_keys_from_parent_map
1556
 
 
1557
 
    def _reload_or_raise(self):
1558
 
        """We just got a NoSuchFile exception.
1559
 
 
1560
 
        Try to reload the indices, if it fails, just raise the current
1561
 
        exception.
1562
 
        """
1563
 
        if self._reload_func is None:
1564
 
            raise
1565
 
        exc_type, exc_value, exc_traceback = sys.exc_info()
1566
 
        trace.mutter('Trying to reload after getting exception: %s',
1567
 
                     exc_value)
1568
 
        if not self._reload_func():
1569
 
            # We tried to reload, but nothing changed, so we fail anyway
1570
 
            trace.mutter('_reload_func indicated nothing has changed.'
1571
 
                         ' Raising original exception.')
1572
 
            raise exc_type, exc_value, exc_traceback
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
 
1117
        return sum((index.key_count() for index in self._indices), 0)
1578
1118
 
1579
1119
    def validate(self):
1580
1120
        """Validate that everything in the index can be accessed."""
1581
 
        while True:
1582
 
            try:
1583
 
                for index in self._indices:
1584
 
                    index.validate()
1585
 
                return
1586
 
            except errors.NoSuchFile:
1587
 
                self._reload_or_raise()
 
1121
        for index in self._indices:
 
1122
            index.validate()
1588
1123
 
1589
1124
 
1590
1125
class InMemoryGraphIndex(GraphIndexBuilder):
1634
1169
            defined order for the result iteration - it will be in the most
1635
1170
            efficient order for the index (keys iteration order in this case).
1636
1171
        """
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]
 
1172
        keys = set(keys)
1641
1173
        if self.reference_lists:
1642
 
            for key in keys:
1643
 
                node = nodes[key]
 
1174
            for key in keys.intersection(self._keys):
 
1175
                node = self._nodes[key]
1644
1176
                if not node[0]:
1645
1177
                    yield self, key, node[2], node[1]
1646
1178
        else:
1647
 
            for key in keys:
1648
 
                node = nodes[key]
 
1179
            for key in keys.intersection(self._keys):
 
1180
                node = self._nodes[key]
1649
1181
                if not node[0]:
1650
1182
                    yield self, key, node[2]
1651
1183
 
1680
1212
                    raise errors.BadIndexKey(key)
1681
1213
                node = self._nodes[key]
1682
1214
                if node[0]:
1683
 
                    continue
 
1215
                    continue 
1684
1216
                if self.reference_lists:
1685
1217
                    yield self, key, node[2], node[1]
1686
1218
                else:
1687
1219
                    yield self, key, node[2]
1688
1220
            return
1689
 
        nodes_by_key = self._get_nodes_by_key()
1690
1221
        for key in keys:
1691
1222
            # sanity check
1692
1223
            if key[0] is None:
1694
1225
            if len(key) != self._key_length:
1695
1226
                raise errors.BadIndexKey(key)
1696
1227
            # find what it refers to:
1697
 
            key_dict = nodes_by_key
 
1228
            key_dict = self._nodes_by_key
1698
1229
            elements = list(key)
1699
1230
            # find the subdict to return
1700
1231
            try:
1711
1242
                    # can't be empty or would not exist
1712
1243
                    item, value = key_dict.iteritems().next()
1713
1244
                    if type(value) == dict:
1714
 
                        # push keys
 
1245
                        # push keys 
1715
1246
                        dicts.extend(key_dict.itervalues())
1716
1247
                    else:
1717
1248
                        # yield keys
1722
1253
 
1723
1254
    def key_count(self):
1724
1255
        """Return an estimate of the number of keys in this index.
1725
 
 
 
1256
        
1726
1257
        For InMemoryGraphIndex the estimate is exact.
1727
1258
        """
1728
 
        return len(self._nodes) - len(self._absent_keys)
 
1259
        return len(self._keys)
1729
1260
 
1730
1261
    def validate(self):
1731
1262
        """In memory index's have no known corruption at the moment."""
1736
1267
 
1737
1268
    Queries against this will emit queries against the adapted Graph with the
1738
1269
    prefix added, queries for all items use iter_entries_prefix. The returned
1739
 
    nodes will have their keys and node references adjusted to remove the
 
1270
    nodes will have their keys and node references adjusted to remove the 
1740
1271
    prefix. Finally, an add_nodes_callback can be supplied - when called the
1741
1272
    nodes and references being added will have prefix prepended.
1742
1273
    """
1769
1300
                    adjusted_references))
1770
1301
        except ValueError:
1771
1302
            # XXX: TODO add an explicit interface for getting the reference list
1772
 
            # status, to handle this bit of user-friendliness in the API more
 
1303
            # status, to handle this bit of user-friendliness in the API more 
1773
1304
            # explicitly.
1774
1305
            for (key, value) in nodes:
1775
1306
                translated_nodes.append((self.prefix + key, value))
1847
1378
 
1848
1379
    def key_count(self):
1849
1380
        """Return an estimate of the number of keys in this index.
1850
 
 
 
1381
        
1851
1382
        For GraphIndexPrefixAdapter this is relatively expensive - key
1852
1383
        iteration with the prefix is done.
1853
1384
        """