~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/btree_index.py

  • Committer: Aaron Bentley
  • Date: 2009-03-24 15:47:32 UTC
  • mto: This revision was merged to the branch mainline in revision 4241.
  • Revision ID: aaron@aaronbentley.com-20090324154732-bwkvi4dx3o90a7dq
Add output, emit minimal inventory delta.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2008, 2009, 2010 Canonical Ltd
 
1
# Copyright (C) 2008 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
17
17
 
18
18
"""B+Tree indices"""
19
19
 
20
 
import cStringIO
 
20
import array
 
21
import bisect
21
22
from bisect import bisect_right
 
23
from copy import deepcopy
22
24
import math
 
25
import struct
23
26
import tempfile
24
27
import zlib
25
28
 
27
30
    chunk_writer,
28
31
    debug,
29
32
    errors,
30
 
    fifo_cache,
31
33
    index,
32
34
    lru_cache,
33
35
    osutils,
34
 
    static_tuple,
35
36
    trace,
36
37
    )
37
38
from bzrlib.index import _OPTION_NODE_REFS, _OPTION_KEY_ELEMENTS, _OPTION_LEN
62
63
    def __init__(self):
63
64
        """Create a _BuilderRow."""
64
65
        self.nodes = 0
65
 
        self.spool = None# tempfile.TemporaryFile(prefix='bzr-index-row-')
 
66
        self.spool = tempfile.TemporaryFile()
66
67
        self.writer = None
67
68
 
68
69
    def finish_node(self, pad=True):
69
70
        byte_lines, _, padding = self.writer.finish()
70
71
        if self.nodes == 0:
71
 
            self.spool = cStringIO.StringIO()
72
72
            # padded note:
73
73
            self.spool.write("\x00" * _RESERVED_HEADER_BYTES)
74
 
        elif self.nodes == 1:
75
 
            # We got bigger than 1 node, switch to a temp file
76
 
            spool = tempfile.TemporaryFile(prefix='bzr-index-row-')
77
 
            spool.write(self.spool.getvalue())
78
 
            self.spool = spool
79
74
        skipped_bytes = 0
80
75
        if not pad and padding:
81
76
            del byte_lines[-1]
160
155
        :param value: The value to associate with the key. It may be any
161
156
            bytes as long as it does not contain \0 or \n.
162
157
        """
163
 
        # Ensure that 'key' is a StaticTuple
164
 
        key = static_tuple.StaticTuple.from_sequence(key).intern()
165
158
        # we don't care about absent_references
166
159
        node_refs, _ = self._check_key_ref_value(key, references, value)
167
160
        if key in self._nodes:
168
161
            raise errors.BadIndexDuplicateKey(key, self)
169
 
        self._nodes[key] = static_tuple.StaticTuple(node_refs, value)
 
162
        self._nodes[key] = (node_refs, value)
 
163
        self._keys.add(key)
170
164
        if self._nodes_by_key is not None and self._key_length > 1:
171
165
            self._update_nodes_by_key(key, value, node_refs)
172
 
        if len(self._nodes) < self._spill_at:
 
166
        if len(self._keys) < self._spill_at:
173
167
            return
174
168
        self._spill_mem_keys_to_disk()
175
169
 
191
185
             backing_pos) = self._spill_mem_keys_and_combine()
192
186
        else:
193
187
            new_backing_file, size = self._spill_mem_keys_without_combining()
 
188
        dir_path, base_name = osutils.split(new_backing_file.name)
194
189
        # Note: The transport here isn't strictly needed, because we will use
195
190
        #       direct access to the new_backing._file object
196
 
        new_backing = BTreeGraphIndex(get_transport('.'), '<temp>', size)
 
191
        new_backing = BTreeGraphIndex(get_transport(dir_path),
 
192
                                      base_name, size)
197
193
        # GC will clean up the file
198
194
        new_backing._file = new_backing_file
199
195
        if self._combine_backing_indices:
204
200
                self._backing_indices[backing_pos] = None
205
201
        else:
206
202
            self._backing_indices.append(new_backing)
 
203
        self._keys = set()
207
204
        self._nodes = {}
208
205
        self._nodes_by_key = None
209
206
 
321
318
            new_row = True
322
319
            for row in reversed(rows[:-1]):
323
320
                # Mark the start of the next node in the node above. If it
324
 
                # doesn't fit then propagate upwards until we find one that
 
321
                # doesn't fit then propogate upwards until we find one that
325
322
                # it does fit into.
326
323
                if row.writer.write(key_line):
327
324
                    row.finish_node()
372
369
        self.row_lengths = []
373
370
        # Loop over all nodes adding them to the bottom row
374
371
        # (rows[-1]). When we finish a chunk in a row,
375
 
        # propagate the key that didn't fit (comes after the chunk) to the
 
372
        # propogate the key that didn't fit (comes after the chunk) to the
376
373
        # row above, transitively.
377
374
        for node in node_iterator:
378
375
            if key_count == 0:
385
382
        for row in reversed(rows):
386
383
            pad = (type(row) != _LeafBuilderRow)
387
384
            row.finish_node(pad=pad)
 
385
        result = tempfile.NamedTemporaryFile(prefix='bzr-index-')
388
386
        lines = [_BTSIGNATURE]
389
387
        lines.append(_OPTION_NODE_REFS + str(self.reference_lists) + '\n')
390
388
        lines.append(_OPTION_KEY_ELEMENTS + str(self._key_length) + '\n')
391
389
        lines.append(_OPTION_LEN + str(key_count) + '\n')
392
390
        row_lengths = [row.nodes for row in rows]
393
391
        lines.append(_OPTION_ROW_LENGTHS + ','.join(map(str, row_lengths)) + '\n')
394
 
        if row_lengths and row_lengths[-1] > 1:
395
 
            result = tempfile.NamedTemporaryFile(prefix='bzr-index-')
396
 
        else:
397
 
            result = cStringIO.StringIO()
398
392
        result.writelines(lines)
399
393
        position = sum(map(len, lines))
400
394
        root_row = True
411
405
            # Special case the first node as it may be prefixed
412
406
            node = row.spool.read(_PAGE_SIZE)
413
407
            result.write(node[reserved:])
414
 
            if len(node) == _PAGE_SIZE:
415
 
                result.write("\x00" * (reserved - position))
 
408
            result.write("\x00" * (reserved - position))
416
409
            position = 0 # Only the root row actually has an offset
417
410
            copied_len = osutils.pumpfile(row.spool, result)
418
411
            if copied_len != (row.nodes - 1) * _PAGE_SIZE:
437
430
    def iter_all_entries(self):
438
431
        """Iterate over all keys within the index
439
432
 
440
 
        :return: An iterable of (index, key, value, reference_lists). There is
441
 
            no defined order for the result iteration - it will be in the most
 
433
        :return: An iterable of (index, key, reference_lists, value). There is no
 
434
            defined order for the result iteration - it will be in the most
442
435
            efficient order for the index (in this case dictionary hash order).
443
436
        """
444
437
        if 'evil' in debug.debug_flags:
463
456
            efficient order for the index (keys iteration order in this case).
464
457
        """
465
458
        keys = set(keys)
466
 
        # Note: We don't use keys.intersection() here. If you read the C api,
467
 
        #       set.intersection(other) special cases when other is a set and
468
 
        #       will iterate the smaller of the two and lookup in the other.
469
 
        #       It does *not* do this for any other type (even dict, unlike
470
 
        #       some other set functions.) Since we expect keys is generally <<
471
 
        #       self._nodes, it is faster to iterate over it in a list
472
 
        #       comprehension
473
 
        nodes = self._nodes
474
 
        local_keys = [key for key in keys if key in nodes]
 
459
        local_keys = keys.intersection(self._keys)
475
460
        if self.reference_lists:
476
461
            for key in local_keys:
477
 
                node = nodes[key]
 
462
                node = self._nodes[key]
478
463
                yield self, key, node[1], node[0]
479
464
        else:
480
465
            for key in local_keys:
481
 
                node = nodes[key]
 
466
                node = self._nodes[key]
482
467
                yield self, key, node[1]
483
468
        # Find things that are in backing indices that have not been handled
484
469
        # yet.
567
552
                    else:
568
553
                        # yield keys
569
554
                        for value in key_dict.itervalues():
570
 
                            yield (self, ) + tuple(value)
 
555
                            yield (self, ) + value
571
556
            else:
572
557
                yield (self, ) + key_dict
573
558
 
594
579
 
595
580
        For InMemoryGraphIndex the estimate is exact.
596
581
        """
597
 
        return len(self._nodes) + sum(backing.key_count() for backing in
 
582
        return len(self._keys) + sum(backing.key_count() for backing in
598
583
            self._backing_indices if backing is not None)
599
584
 
600
585
    def validate(self):
604
589
class _LeafNode(object):
605
590
    """A leaf node for a serialised B+Tree index."""
606
591
 
607
 
    __slots__ = ('keys', 'min_key', 'max_key')
608
 
 
609
592
    def __init__(self, bytes, key_length, ref_list_length):
610
593
        """Parse bytes to create a leaf node object."""
611
594
        # splitlines mangles the \r delimiters.. don't use it.
612
 
        key_list = _btree_serializer._parse_leaf_lines(bytes,
613
 
            key_length, ref_list_length)
614
 
        if key_list:
615
 
            self.min_key = key_list[0][0]
616
 
            self.max_key = key_list[-1][0]
617
 
        else:
618
 
            self.min_key = self.max_key = None
619
 
        self.keys = dict(key_list)
 
595
        self.keys = dict(_btree_serializer._parse_leaf_lines(bytes,
 
596
            key_length, ref_list_length))
620
597
 
621
598
 
622
599
class _InternalNode(object):
623
600
    """An internal node for a serialised B+Tree index."""
624
601
 
625
 
    __slots__ = ('keys', 'offset')
626
 
 
627
602
    def __init__(self, bytes):
628
603
        """Parse bytes to create an internal node object."""
629
604
        # splitlines mangles the \r delimiters.. don't use it.
632
607
    def _parse_lines(self, lines):
633
608
        nodes = []
634
609
        self.offset = int(lines[1][7:])
635
 
        as_st = static_tuple.StaticTuple.from_sequence
636
610
        for line in lines[2:]:
637
611
            if line == '':
638
612
                break
639
 
            nodes.append(as_st(map(intern, line.split('\0'))).intern())
 
613
            nodes.append(tuple(line.split('\0')))
640
614
        return nodes
641
615
 
642
616
 
647
621
    memory except when very large walks are done.
648
622
    """
649
623
 
650
 
    def __init__(self, transport, name, size, unlimited_cache=False,
651
 
                 offset=0):
 
624
    def __init__(self, transport, name, size):
652
625
        """Create a B+Tree index object on the index name.
653
626
 
654
627
        :param transport: The transport to read data for the index from.
658
631
            the initial read (to read the root node header) can be done
659
632
            without over-reading even on empty indices, and on small indices
660
633
            allows single-IO to read the entire index.
661
 
        :param unlimited_cache: If set to True, then instead of using an
662
 
            LRUCache with size _NODE_CACHE_SIZE, we will use a dict and always
663
 
            cache all leaf nodes.
664
 
        :param offset: The start of the btree index data isn't byte 0 of the
665
 
            file. Instead it starts at some point later.
666
634
        """
667
635
        self._transport = transport
668
636
        self._name = name
670
638
        self._file = None
671
639
        self._recommended_pages = self._compute_recommended_pages()
672
640
        self._root_node = None
673
 
        self._base_offset = offset
674
641
        # Default max size is 100,000 leave values
675
642
        self._leaf_value_cache = None # lru_cache.LRUCache(100*1000)
676
 
        if unlimited_cache:
677
 
            self._leaf_node_cache = {}
678
 
            self._internal_node_cache = {}
679
 
        else:
680
 
            self._leaf_node_cache = lru_cache.LRUCache(_NODE_CACHE_SIZE)
681
 
            # We use a FIFO here just to prevent possible blowout. However, a
682
 
            # 300k record btree has only 3k leaf nodes, and only 20 internal
683
 
            # nodes. A value of 100 scales to ~100*100*100 = 1M records.
684
 
            self._internal_node_cache = fifo_cache.FIFOCache(100)
 
643
        self._leaf_node_cache = lru_cache.LRUCache(_NODE_CACHE_SIZE)
 
644
        self._internal_node_cache = lru_cache.LRUCache()
685
645
        self._key_count = None
686
646
        self._row_lengths = None
687
647
        self._row_offsets = None # Start of each row, [-1] is the end
719
679
                if start_of_leaves is None:
720
680
                    start_of_leaves = self._row_offsets[-2]
721
681
                if node_pos < start_of_leaves:
722
 
                    self._internal_node_cache[node_pos] = node
 
682
                    self._internal_node_cache.add(node_pos, node)
723
683
                else:
724
 
                    self._leaf_node_cache[node_pos] = node
 
684
                    self._leaf_node_cache.add(node_pos, node)
725
685
            found[node_pos] = node
726
686
        return found
727
687
 
866
826
            new_tips = next_tips
867
827
        return final_offsets
868
828
 
869
 
    def clear_cache(self):
870
 
        """Clear out any cached/memoized values.
871
 
 
872
 
        This can be called at any time, but generally it is used when we have
873
 
        extracted some information, but don't expect to be requesting any more
874
 
        from this index.
875
 
        """
876
 
        # Note that we don't touch self._root_node or self._internal_node_cache
877
 
        # We don't expect either of those to be big, and it can save
878
 
        # round-trips in the future. We may re-evaluate this if InternalNode
879
 
        # memory starts to be an issue.
880
 
        self._leaf_node_cache.clear()
881
 
 
882
829
    def external_references(self, ref_list_num):
883
830
        if self._root_node is None:
884
831
            self._get_root_node()
1087
1034
            output.append(cur_out)
1088
1035
        return output
1089
1036
 
1090
 
    def _walk_through_internal_nodes(self, keys):
1091
 
        """Take the given set of keys, and find the corresponding LeafNodes.
1092
 
 
1093
 
        :param keys: An unsorted iterable of keys to search for
1094
 
        :return: (nodes, index_and_keys)
1095
 
            nodes is a dict mapping {index: LeafNode}
1096
 
            keys_at_index is a list of tuples of [(index, [keys for Leaf])]
1097
 
        """
1098
 
        # 6 seconds spent in miss_torture using the sorted() line.
1099
 
        # Even with out of order disk IO it seems faster not to sort it when
1100
 
        # large queries are being made.
1101
 
        keys_at_index = [(0, sorted(keys))]
1102
 
 
1103
 
        for row_pos, next_row_start in enumerate(self._row_offsets[1:-1]):
1104
 
            node_indexes = [idx for idx, s_keys in keys_at_index]
1105
 
            nodes = self._get_internal_nodes(node_indexes)
1106
 
 
1107
 
            next_nodes_and_keys = []
1108
 
            for node_index, sub_keys in keys_at_index:
1109
 
                node = nodes[node_index]
1110
 
                positions = self._multi_bisect_right(sub_keys, node.keys)
1111
 
                node_offset = next_row_start + node.offset
1112
 
                next_nodes_and_keys.extend([(node_offset + pos, s_keys)
1113
 
                                           for pos, s_keys in positions])
1114
 
            keys_at_index = next_nodes_and_keys
1115
 
        # We should now be at the _LeafNodes
1116
 
        node_indexes = [idx for idx, s_keys in keys_at_index]
1117
 
 
1118
 
        # TODO: We may *not* want to always read all the nodes in one
1119
 
        #       big go. Consider setting a max size on this.
1120
 
        nodes = self._get_leaf_nodes(node_indexes)
1121
 
        return nodes, keys_at_index
1122
 
 
1123
1037
    def iter_entries(self, keys):
1124
1038
        """Iterate over keys within the index.
1125
1039
 
1163
1077
        needed_keys = keys
1164
1078
        if not needed_keys:
1165
1079
            return
1166
 
        nodes, nodes_and_keys = self._walk_through_internal_nodes(needed_keys)
 
1080
        # 6 seconds spent in miss_torture using the sorted() line.
 
1081
        # Even with out of order disk IO it seems faster not to sort it when
 
1082
        # large queries are being made.
 
1083
        needed_keys = sorted(needed_keys)
 
1084
 
 
1085
        nodes_and_keys = [(0, needed_keys)]
 
1086
 
 
1087
        for row_pos, next_row_start in enumerate(self._row_offsets[1:-1]):
 
1088
            node_indexes = [idx for idx, s_keys in nodes_and_keys]
 
1089
            nodes = self._get_internal_nodes(node_indexes)
 
1090
 
 
1091
            next_nodes_and_keys = []
 
1092
            for node_index, sub_keys in nodes_and_keys:
 
1093
                node = nodes[node_index]
 
1094
                positions = self._multi_bisect_right(sub_keys, node.keys)
 
1095
                node_offset = next_row_start + node.offset
 
1096
                next_nodes_and_keys.extend([(node_offset + pos, s_keys)
 
1097
                                           for pos, s_keys in positions])
 
1098
            nodes_and_keys = next_nodes_and_keys
 
1099
        # We should now be at the _LeafNodes
 
1100
        node_indexes = [idx for idx, s_keys in nodes_and_keys]
 
1101
 
 
1102
        # TODO: We may *not* want to always read all the nodes in one
 
1103
        #       big go. Consider setting a max size on this.
 
1104
 
 
1105
        nodes = self._get_leaf_nodes(node_indexes)
1167
1106
        for node_index, sub_keys in nodes_and_keys:
1168
1107
            if not sub_keys:
1169
1108
                continue
1176
1115
                    else:
1177
1116
                        yield (self, next_sub_key, value)
1178
1117
 
1179
 
    def _find_ancestors(self, keys, ref_list_num, parent_map, missing_keys):
1180
 
        """Find the parent_map information for the set of keys.
1181
 
 
1182
 
        This populates the parent_map dict and missing_keys set based on the
1183
 
        queried keys. It also can fill out an arbitrary number of parents that
1184
 
        it finds while searching for the supplied keys.
1185
 
 
1186
 
        It is unlikely that you want to call this directly. See
1187
 
        "CombinedGraphIndex.find_ancestry()" for a more appropriate API.
1188
 
 
1189
 
        :param keys: A keys whose ancestry we want to return
1190
 
            Every key will either end up in 'parent_map' or 'missing_keys'.
1191
 
        :param ref_list_num: This index in the ref_lists is the parents we
1192
 
            care about.
1193
 
        :param parent_map: {key: parent_keys} for keys that are present in this
1194
 
            index. This may contain more entries than were in 'keys', that are
1195
 
            reachable ancestors of the keys requested.
1196
 
        :param missing_keys: keys which are known to be missing in this index.
1197
 
            This may include parents that were not directly requested, but we
1198
 
            were able to determine that they are not present in this index.
1199
 
        :return: search_keys    parents that were found but not queried to know
1200
 
            if they are missing or present. Callers can re-query this index for
1201
 
            those keys, and they will be placed into parent_map or missing_keys
1202
 
        """
1203
 
        if not self.key_count():
1204
 
            # We use key_count() to trigger reading the root node and
1205
 
            # determining info about this BTreeGraphIndex
1206
 
            # If we don't have any keys, then everything is missing
1207
 
            missing_keys.update(keys)
1208
 
            return set()
1209
 
        if ref_list_num >= self.node_ref_lists:
1210
 
            raise ValueError('No ref list %d, index has %d ref lists'
1211
 
                % (ref_list_num, self.node_ref_lists))
1212
 
 
1213
 
        # The main trick we are trying to accomplish is that when we find a
1214
 
        # key listing its parents, we expect that the parent key is also likely
1215
 
        # to sit on the same page. Allowing us to expand parents quickly
1216
 
        # without suffering the full stack of bisecting, etc.
1217
 
        nodes, nodes_and_keys = self._walk_through_internal_nodes(keys)
1218
 
 
1219
 
        # These are parent keys which could not be immediately resolved on the
1220
 
        # page where the child was present. Note that we may already be
1221
 
        # searching for that key, and it may actually be present [or known
1222
 
        # missing] on one of the other pages we are reading.
1223
 
        # TODO:
1224
 
        #   We could try searching for them in the immediate previous or next
1225
 
        #   page. If they occur "later" we could put them in a pending lookup
1226
 
        #   set, and then for each node we read thereafter we could check to
1227
 
        #   see if they are present.
1228
 
        #   However, we don't know the impact of keeping this list of things
1229
 
        #   that I'm going to search for every node I come across from here on
1230
 
        #   out.
1231
 
        #   It doesn't handle the case when the parent key is missing on a
1232
 
        #   page that we *don't* read. So we already have to handle being
1233
 
        #   re-entrant for that.
1234
 
        #   Since most keys contain a date string, they are more likely to be
1235
 
        #   found earlier in the file than later, but we would know that right
1236
 
        #   away (key < min_key), and wouldn't keep searching it on every other
1237
 
        #   page that we read.
1238
 
        #   Mostly, it is an idea, one which should be benchmarked.
1239
 
        parents_not_on_page = set()
1240
 
 
1241
 
        for node_index, sub_keys in nodes_and_keys:
1242
 
            if not sub_keys:
1243
 
                continue
1244
 
            # sub_keys is all of the keys we are looking for that should exist
1245
 
            # on this page, if they aren't here, then they won't be found
1246
 
            node = nodes[node_index]
1247
 
            node_keys = node.keys
1248
 
            parents_to_check = set()
1249
 
            for next_sub_key in sub_keys:
1250
 
                if next_sub_key not in node_keys:
1251
 
                    # This one is just not present in the index at all
1252
 
                    missing_keys.add(next_sub_key)
1253
 
                else:
1254
 
                    value, refs = node_keys[next_sub_key]
1255
 
                    parent_keys = refs[ref_list_num]
1256
 
                    parent_map[next_sub_key] = parent_keys
1257
 
                    parents_to_check.update(parent_keys)
1258
 
            # Don't look for things we've already found
1259
 
            parents_to_check = parents_to_check.difference(parent_map)
1260
 
            # this can be used to test the benefit of having the check loop
1261
 
            # inlined.
1262
 
            # parents_not_on_page.update(parents_to_check)
1263
 
            # continue
1264
 
            while parents_to_check:
1265
 
                next_parents_to_check = set()
1266
 
                for key in parents_to_check:
1267
 
                    if key in node_keys:
1268
 
                        value, refs = node_keys[key]
1269
 
                        parent_keys = refs[ref_list_num]
1270
 
                        parent_map[key] = parent_keys
1271
 
                        next_parents_to_check.update(parent_keys)
1272
 
                    else:
1273
 
                        # This parent either is genuinely missing, or should be
1274
 
                        # found on another page. Perf test whether it is better
1275
 
                        # to check if this node should fit on this page or not.
1276
 
                        # in the 'everything-in-one-pack' scenario, this *not*
1277
 
                        # doing the check is 237ms vs 243ms.
1278
 
                        # So slightly better, but I assume the standard 'lots
1279
 
                        # of packs' is going to show a reasonable improvement
1280
 
                        # from the check, because it avoids 'going around
1281
 
                        # again' for everything that is in another index
1282
 
                        # parents_not_on_page.add(key)
1283
 
                        # Missing for some reason
1284
 
                        if key < node.min_key:
1285
 
                            # in the case of bzr.dev, 3.4k/5.3k misses are
1286
 
                            # 'earlier' misses (65%)
1287
 
                            parents_not_on_page.add(key)
1288
 
                        elif key > node.max_key:
1289
 
                            # This parent key would be present on a different
1290
 
                            # LeafNode
1291
 
                            parents_not_on_page.add(key)
1292
 
                        else:
1293
 
                            # assert key != node.min_key and key != node.max_key
1294
 
                            # If it was going to be present, it would be on
1295
 
                            # *this* page, so mark it missing.
1296
 
                            missing_keys.add(key)
1297
 
                parents_to_check = next_parents_to_check.difference(parent_map)
1298
 
                # Might want to do another .difference() from missing_keys
1299
 
        # parents_not_on_page could have been found on a different page, or be
1300
 
        # known to be missing. So cull out everything that has already been
1301
 
        # found.
1302
 
        search_keys = parents_not_on_page.difference(
1303
 
            parent_map).difference(missing_keys)
1304
 
        return search_keys
1305
 
 
1306
1118
    def iter_entries_prefix(self, keys):
1307
1119
        """Iterate over keys within the index using prefix matching.
1308
1120
 
1498
1310
        # list of (offset, length) regions of the file that should, evenually
1499
1311
        # be read in to data_ranges, either from 'bytes' or from the transport
1500
1312
        ranges = []
1501
 
        base_offset = self._base_offset
1502
1313
        for index in nodes:
1503
 
            offset = (index * _PAGE_SIZE)
 
1314
            offset = index * _PAGE_SIZE
1504
1315
            size = _PAGE_SIZE
1505
1316
            if index == 0:
1506
1317
                # Root node - special case
1510
1321
                    # The only case where we don't know the size, is for very
1511
1322
                    # small indexes. So we read the whole thing
1512
1323
                    bytes = self._transport.get_bytes(self._name)
1513
 
                    num_bytes = len(bytes)
1514
 
                    self._size = num_bytes - base_offset
 
1324
                    self._size = len(bytes)
1515
1325
                    # the whole thing should be parsed out of 'bytes'
1516
 
                    ranges = [(start, min(_PAGE_SIZE, num_bytes - start))
1517
 
                        for start in xrange(base_offset, num_bytes, _PAGE_SIZE)]
 
1326
                    ranges.append((0, len(bytes)))
1518
1327
                    break
1519
1328
            else:
1520
1329
                if offset > self._size:
1522
1331
                                         ' of the file %s > %s'
1523
1332
                                         % (offset, self._size))
1524
1333
                size = min(size, self._size - offset)
1525
 
            ranges.append((base_offset + offset, size))
 
1334
            ranges.append((offset, size))
1526
1335
        if not ranges:
1527
1336
            return
1528
1337
        elif bytes is not None:
1529
1338
            # already have the whole file
1530
 
            data_ranges = [(start, bytes[start:start+size])
1531
 
                           for start, size in ranges]
 
1339
            data_ranges = [(start, bytes[start:start+_PAGE_SIZE])
 
1340
                           for start in xrange(0, len(bytes), _PAGE_SIZE)]
1532
1341
        elif self._file is None:
1533
1342
            data_ranges = self._transport.readv(self._name, ranges)
1534
1343
        else:
1537
1346
                self._file.seek(offset)
1538
1347
                data_ranges.append((offset, self._file.read(size)))
1539
1348
        for offset, data in data_ranges:
1540
 
            offset -= base_offset
1541
1349
            if offset == 0:
1542
1350
                # extract the header
1543
1351
                offset, data = self._parse_header_from_bytes(data)
1571
1379
 
1572
1380
 
1573
1381
try:
1574
 
    from bzrlib import _btree_serializer_pyx as _btree_serializer
1575
 
except ImportError, e:
1576
 
    osutils.failed_to_load_extension(e)
 
1382
    from bzrlib import _btree_serializer_c as _btree_serializer
 
1383
except ImportError:
1577
1384
    from bzrlib import _btree_serializer_py as _btree_serializer