/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
1
# Copyright (C) 2008 Canonical Ltd
2
#
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
7
#
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
12
#
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
17
"""Persistent maps from tuple_of_strings->string using CHK stores.
18
19
Overview and current status:
20
21
The CHKMap class implements a dict from tuple_of_strings->string by using a trie
22
with internal nodes of 8-bit fan out; The key tuples are mapped to strings by
23
joining them by \x00, and \x00 padding shorter keys out to the length of the
24
longest key. Leaf nodes are packed as densely as possible, and internal nodes
3735.11.1 by John Arbash Meinel
Clean up some trailing whitespace.
25
are all and additional 8-bits wide leading to a sparse upper tree.
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
26
27
Updates to a CHKMap are done preferentially via the apply_delta method, to
28
allow optimisation of the update operation; but individual map/unmap calls are
29
possible and supported. All changes via map/unmap are buffered in memory until
30
the _save method is called to force serialisation of the tree. apply_delta
31
performs a _save implicitly.
32
33
TODO:
34
-----
35
36
Densely packed upper nodes.
37
38
"""
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
39
3735.2.31 by Robert Collins
CHKMap.iter_changes
40
import heapq
3735.2.62 by Robert Collins
Create a rudimentary CHK page cache.
41
3735.9.18 by John Arbash Meinel
Make the versionedfile import lazy.
42
from bzrlib import lazy_import
43
lazy_import.lazy_import(globals(), """
3735.16.3 by John Arbash Meinel
Add functions for _search_key_16 and _search_key_255 and some basic tests for them.
44
import zlib
45
import struct
46
3735.9.18 by John Arbash Meinel
Make the versionedfile import lazy.
47
from bzrlib import versionedfile
48
""")
3735.16.7 by John Arbash Meinel
Start parameterizing CHKInventory and CHKSerializer so that we can
49
from bzrlib import (
50
    lru_cache,
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
51
    osutils,
3735.16.7 by John Arbash Meinel
Start parameterizing CHKInventory and CHKSerializer so that we can
52
    registry,
53
    )
3735.2.62 by Robert Collins
Create a rudimentary CHK page cache.
54
55
# approx 2MB
3735.14.5 by John Arbash Meinel
Change _check_remap to only page in a batch of children at a time.
56
# If each line is 50 bytes, and you have 255 internal pages, with 255-way fan
57
# out, it takes 3.1MB to cache the layer.
58
_PAGE_CACHE_SIZE = 4*1024*1024
3735.14.2 by John Arbash Meinel
Finish using the page cache as part of _check_remap, add debugging functions
59
# We are caching bytes so len(value) is perfectly accurate
60
_page_cache = lru_cache.LRUSizeCache(_PAGE_CACHE_SIZE)
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
61
62
3735.16.6 by John Arbash Meinel
Include a _search_key_plain function.
63
def _search_key_plain(key):
64
    """Map the key tuple into a search string that just uses the key bytes."""
65
    return '\x00'.join(key)
66
67
3735.2.87 by Vincent Ladeuil
Same player shoots again, zlib.crc32, we'll get you.
68
def _crc32(bit):
3735.2.83 by Vincent Ladeuil
Better fix with explanation for zlib.crc32.
69
    # Depending on python version and platform, zlib.crc32 will return either a
70
    # signed (<= 2.5 >= 3.0) or an unsigned (2.5, 2.6).
71
    # http://docs.python.org/library/zlib.html recommends using a mask to force
72
    # an unsigned value to ensure the same numeric value (unsigned) is obtained
73
    # across all python versions and platforms.
3735.2.84 by John Arbash Meinel
Comment about using using 0xFFFFFFFF as part of _search_key_255
74
    # Note: However, on 32-bit platforms this causes an upcast to PyLong, which
75
    #       are generally slower than PyInts. However, if performance becomes
76
    #       critical, we should probably write the whole thing as an extension
77
    #       anyway.
78
    #       Though we really don't need that 32nd bit of accuracy. (even 2**24
79
    #       is probably enough node fan out for realistic trees.)
3735.2.87 by Vincent Ladeuil
Same player shoots again, zlib.crc32, we'll get you.
80
    return zlib.crc32(bit)&0xFFFFFFFF
81
82
83
def _search_key_16(key):
84
    """Map the key tuple into a search key string which has 16-way fan out."""
85
    return '\x00'.join(['%08X' % _crc32(bit) for bit in key])
86
87
88
def _search_key_255(key):
89
    """Map the key tuple into a search key string which has 255-way fan out.
90
91
    We use 255-way because '\n' is used as a delimiter, and causes problems
92
    while parsing.
93
    """
94
    bytes = '\x00'.join([struct.pack('>L', _crc32(bit)) for bit in key])
3735.16.3 by John Arbash Meinel
Add functions for _search_key_16 and _search_key_255 and some basic tests for them.
95
    return bytes.replace('\n', '_')
96
97
3735.16.7 by John Arbash Meinel
Start parameterizing CHKInventory and CHKSerializer so that we can
98
search_key_registry = registry.Registry()
99
search_key_registry.register('plain', _search_key_plain)
100
search_key_registry.register('hash-16-way', _search_key_16)
101
search_key_registry.register('hash-255-way', _search_key_255)
102
103
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
104
class CHKMap(object):
105
    """A persistent map from string to string backed by a CHK store."""
106
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
107
    def __init__(self, store, root_key, search_key_func=None):
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
108
        """Create a CHKMap object.
109
110
        :param store: The store the CHKMap is stored in.
111
        :param root_key: The root key of the map. None to create an empty
112
            CHKMap.
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
113
        :param search_key_func: A function mapping a key => bytes. These bytes
114
            are then used by the internal nodes to split up leaf nodes into
115
            multiple pages.
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
116
        """
117
        self._store = store
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
118
        if search_key_func is None:
3735.16.6 by John Arbash Meinel
Include a _search_key_plain function.
119
            search_key_func = _search_key_plain
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
120
        self._search_key_func = search_key_func
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
121
        if root_key is None:
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
122
            self._root_node = LeafNode(search_key_func=search_key_func)
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
123
        else:
3735.2.41 by Robert Collins
Make the parent_id_basename index be updated during CHKInventory.apply_delta.
124
            self._root_node = self._node_key(root_key)
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
125
126
    def apply_delta(self, delta):
127
        """Apply a delta to the map.
128
129
        :param delta: An iterable of old_key, new_key, new_value tuples.
130
            If new_key is not None, then new_key->new_value is inserted
131
            into the map; if old_key is not None, then the old mapping
132
            of old_key is removed.
133
        """
134
        for old, new, value in delta:
135
            if old is not None and old != new:
136
                # unmap
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
137
                self.unmap(old)
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
138
        for old, new, value in delta:
139
            if new is not None:
140
                # map
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
141
                self.map(new, value)
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
142
        return self._save()
143
144
    def _ensure_root(self):
145
        """Ensure that the root node is an object not a key."""
146
        if type(self._root_node) == tuple:
147
            # Demand-load the root
3735.2.31 by Robert Collins
CHKMap.iter_changes
148
            self._root_node = self._get_node(self._root_node)
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
149
            # XXX: Shouldn't this be put into _deserialize?
150
            self._root_node._search_key_func = self._search_key_func
3735.2.31 by Robert Collins
CHKMap.iter_changes
151
152
    def _get_node(self, node):
153
        """Get a node.
154
155
        Node that this does not update the _items dict in objects containing a
156
        reference to this node. As such it does not prevent subsequent IO being
157
        performed.
3735.11.1 by John Arbash Meinel
Clean up some trailing whitespace.
158
3735.2.31 by Robert Collins
CHKMap.iter_changes
159
        :param node: A tuple key or node object.
160
        :return: A node object.
161
        """
162
        if type(node) == tuple:
163
            bytes = self._read_bytes(node)
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
164
            return _deserialise(bytes, node,
165
                search_key_func=self._search_key_func)
3735.2.31 by Robert Collins
CHKMap.iter_changes
166
        else:
167
            return node
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
168
169
    def _read_bytes(self, key):
170
        stream = self._store.get_record_stream([key], 'unordered', True)
171
        return stream.next().get_bytes_as('fulltext')
172
3735.15.16 by John Arbash Meinel
Properly fix up the dump_tree tests, we now suppress the keys by default.
173
    def _dump_tree(self, include_keys=False):
3735.9.2 by John Arbash Meinel
Add a _dump_tree helper that assists in debugging what is going on.
174
        """Return the tree in a string representation."""
175
        self._ensure_root()
3735.15.15 by John Arbash Meinel
Change child_child to use _dump_tree,
176
        res = self._dump_tree_node(self._root_node, prefix='', indent='',
177
                                   include_keys=include_keys)
3735.11.9 by John Arbash Meinel
Switch _dump_tree to returning trailing '\n' for nicer results
178
        res.append('') # Give a trailing '\n'
3735.9.4 by John Arbash Meinel
Some small cleanups, and fix _dump_tree to handle in-progress nodes.
179
        return '\n'.join(res)
3735.9.2 by John Arbash Meinel
Add a _dump_tree helper that assists in debugging what is going on.
180
3735.15.15 by John Arbash Meinel
Change child_child to use _dump_tree,
181
    def _dump_tree_node(self, node, prefix, indent, include_keys=True):
3735.9.2 by John Arbash Meinel
Add a _dump_tree helper that assists in debugging what is going on.
182
        """For this node and all children, generate a string representation."""
183
        result = []
3735.15.15 by John Arbash Meinel
Change child_child to use _dump_tree,
184
        if not include_keys:
185
            key_str = ''
186
        else:
187
            node_key = node.key()
188
            if node_key is not None:
189
                key_str = ' %s' % (node_key[0],)
190
            else:
191
                key_str = ' None'
192
        result.append('%s%r %s%s' % (indent, prefix, node.__class__.__name__,
193
                                     key_str))
3735.9.2 by John Arbash Meinel
Add a _dump_tree helper that assists in debugging what is going on.
194
        if isinstance(node, InternalNode):
195
            # Trigger all child nodes to get loaded
196
            list(node._iter_nodes(self._store))
3735.9.4 by John Arbash Meinel
Some small cleanups, and fix _dump_tree to handle in-progress nodes.
197
            for prefix, sub in sorted(node._items.iteritems()):
3735.15.15 by John Arbash Meinel
Change child_child to use _dump_tree,
198
                result.extend(self._dump_tree_node(sub, prefix, indent + '  ',
199
                                                   include_keys=include_keys))
3735.9.2 by John Arbash Meinel
Add a _dump_tree helper that assists in debugging what is going on.
200
        else:
3735.9.4 by John Arbash Meinel
Some small cleanups, and fix _dump_tree to handle in-progress nodes.
201
            for key, value in sorted(node._items.iteritems()):
202
                result.append('      %r %r' % (key, value))
3735.9.2 by John Arbash Meinel
Add a _dump_tree helper that assists in debugging what is going on.
203
        return result
204
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
205
    @classmethod
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
206
    def from_dict(klass, store, initial_value, maximum_size=0, key_width=1):
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
207
        """Create a CHKMap in store with initial_value as the content.
3735.11.1 by John Arbash Meinel
Clean up some trailing whitespace.
208
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
209
        :param store: The store to record initial_value in, a VersionedFiles
210
            object with 1-tuple keys supporting CHK key generation.
211
        :param initial_value: A dict to store in store. Its keys and values
212
            must be bytestrings.
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
213
        :param maximum_size: The maximum_size rule to apply to nodes. This
214
            determines the size at which no new data is added to a single node.
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
215
        :param key_width: The number of elements in each key_tuple being stored
216
            in this map.
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
217
        :return: The root chk of te resulting CHKMap.
218
        """
219
        result = CHKMap(store, None)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
220
        result._root_node.set_maximum_size(maximum_size)
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
221
        result._root_node._key_width = key_width
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
222
        delta = []
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
223
        for key, value in initial_value.items():
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
224
            delta.append((None, key, value))
225
        result.apply_delta(delta)
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
226
        return result._save()
227
3735.2.30 by Robert Collins
Start iter_changes between CHKMap instances.
228
    def iter_changes(self, basis):
229
        """Iterate over the changes between basis and self.
230
231
        :return: An iterator of tuples: (key, old_value, new_value). Old_value
232
            is None for keys only in self; new_value is None for keys only in
233
            basis.
234
        """
3735.2.31 by Robert Collins
CHKMap.iter_changes
235
        # Overview:
236
        # Read both trees in lexographic, highest-first order.
237
        # Any identical nodes we skip
238
        # Any unique prefixes we output immediately.
239
        # values in a leaf node are treated as single-value nodes in the tree
240
        # which allows them to be not-special-cased. We know to output them
241
        # because their value is a string, not a key(tuple) or node.
242
        #
243
        # corner cases to beware of when considering this function:
244
        # *) common references are at different heights.
245
        #    consider two trees:
246
        #    {'a': LeafNode={'aaa':'foo', 'aab':'bar'}, 'b': LeafNode={'b'}}
247
        #    {'a': InternalNode={'aa':LeafNode={'aaa':'foo', 'aab':'bar'}, 'ab':LeafNode={'ab':'bar'}}
248
        #     'b': LeafNode={'b'}}
249
        #    the node with aaa/aab will only be encountered in the second tree
250
        #    after reading the 'a' subtree, but it is encountered in the first
251
        #    tree immediately. Variations on this may have read internal nodes like this.
252
        #    we want to cut the entire pending subtree when we realise we have a common node.
3735.11.1 by John Arbash Meinel
Clean up some trailing whitespace.
253
        #    For this we use a list of keys - the path to a node - and check the entire path is
3735.2.31 by Robert Collins
CHKMap.iter_changes
254
        #    clean as we process each item.
255
        if self._node_key(self._root_node) == self._node_key(basis._root_node):
256
            return
257
        self._ensure_root()
258
        basis._ensure_root()
259
        excluded_keys = set()
260
        self_node = self._root_node
261
        basis_node = basis._root_node
262
        # A heap, each element is prefix, node(tuple/NodeObject/string),
263
        # key_path (a list of tuples, tail-sharing down the tree.)
264
        self_pending = []
265
        basis_pending = []
266
        def process_node(prefix, node, path, a_map, pending):
267
            # take a node and expand it
268
            node = a_map._get_node(node)
269
            if type(node) == LeafNode:
270
                path = (node._key, path)
271
                for key, value in node._items.items():
272
                    heapq.heappush(pending, ('\x00'.join(key), value, path))
273
            else:
274
                # type(node) == InternalNode
275
                path = (node._key, path)
276
                for prefix, child in node._items.items():
277
                    heapq.heappush(pending, (prefix, child, path))
278
        process_node(None, self_node, None, self, self_pending)
279
        process_node(None, basis_node, None, basis, basis_pending)
280
        self_seen = set()
281
        basis_seen = set()
282
        excluded_keys = set()
283
        def check_excluded(key_path):
284
            # Note that this is N^2, it depends on us trimming trees
285
            # aggressively to not become slow.
286
            # A better implementation would probably have a reverse map
3735.11.1 by John Arbash Meinel
Clean up some trailing whitespace.
287
            # back to the children of a node, and jump straight to it when
3735.2.31 by Robert Collins
CHKMap.iter_changes
288
            # a common node is detected, the proceed to remove the already
289
            # pending children. bzrlib.graph has a searcher module with a
290
            # similar problem.
291
            while key_path is not None:
292
                key, key_path = key_path
293
                if key in excluded_keys:
294
                    return True
295
            return False
296
3735.2.32 by Robert Collins
Activate test for common node skipping. - 50 times performance improvement.
297
        loop_counter = 0
3735.2.31 by Robert Collins
CHKMap.iter_changes
298
        while self_pending or basis_pending:
3735.2.32 by Robert Collins
Activate test for common node skipping. - 50 times performance improvement.
299
            loop_counter += 1
3735.2.31 by Robert Collins
CHKMap.iter_changes
300
            if not self_pending:
301
                # self is exhausted: output remainder of basis
302
                for prefix, node, path in basis_pending:
303
                    if check_excluded(path):
304
                        continue
305
                    node = basis._get_node(node)
306
                    if type(node) == str:
307
                        # a value
308
                        yield (tuple(prefix.split('\x00')), node, None)
309
                    else:
310
                        # subtree - fastpath the entire thing.
311
                        for key, value in node.iteritems(basis._store):
312
                            yield (key, value, None)
313
                return
314
            elif not basis_pending:
315
                # basis is exhausted: output remainder of self.
316
                for prefix, node, path in self_pending:
317
                    if check_excluded(path):
318
                        continue
319
                    node = self._get_node(node)
320
                    if type(node) == str:
321
                        # a value
322
                        yield (tuple(prefix.split('\x00')), None, node)
323
                    else:
324
                        # subtree - fastpath the entire thing.
325
                        for key, value in node.iteritems(self._store):
326
                            yield (key, None, value)
327
                return
328
            else:
329
                # XXX: future optimisation - yield the smaller items
330
                # immediately rather than pushing everything on/off the
331
                # heaps. Applies to both internal nodes and leafnodes.
332
                if self_pending[0][0] < basis_pending[0][0]:
333
                    # expand self
334
                    prefix, node, path = heapq.heappop(self_pending)
335
                    if check_excluded(path):
336
                        continue
337
                    if type(node) == str:
338
                        # a value
339
                        yield (tuple(prefix.split('\x00')), None, node)
340
                    else:
341
                        process_node(prefix, node, path, self, self_pending)
342
                        continue
343
                elif self_pending[0][0] > basis_pending[0][0]:
344
                    # expand basis
345
                    prefix, node, path = heapq.heappop(basis_pending)
346
                    if check_excluded(path):
347
                        continue
348
                    if type(node) == str:
349
                        # a value
350
                        yield (tuple(prefix.split('\x00')), node, None)
351
                    else:
352
                        process_node(prefix, node, path, basis, basis_pending)
353
                        continue
354
                else:
355
                    # common prefix: possibly expand both
356
                    if type(self_pending[0][1]) != str:
357
                        # process next self
358
                        read_self = True
359
                    else:
360
                        read_self = False
361
                    if type(basis_pending[0][1]) != str:
362
                        # process next basis
363
                        read_basis = True
364
                    else:
365
                        read_basis = False
366
                    if not read_self and not read_basis:
367
                        # compare a common value
368
                        self_details = heapq.heappop(self_pending)
369
                        basis_details = heapq.heappop(basis_pending)
370
                        if self_details[1] != basis_details[1]:
371
                            yield (tuple(self_details[0].split('\x00')),
372
                                basis_details[1], self_details[1])
373
                        continue
3735.2.32 by Robert Collins
Activate test for common node skipping. - 50 times performance improvement.
374
                    # At least one side wasn't a string.
375
                    if (self._node_key(self_pending[0][1]) ==
376
                        self._node_key(basis_pending[0][1])):
377
                        # Identical pointers, skip (and don't bother adding to
378
                        # excluded, it won't turn up again.
379
                        heapq.heappop(self_pending)
380
                        heapq.heappop(basis_pending)
381
                        continue
382
                    # Now we need to expand this node before we can continue
3735.2.31 by Robert Collins
CHKMap.iter_changes
383
                    if read_self:
384
                        prefix, node, path = heapq.heappop(self_pending)
385
                        if check_excluded(path):
386
                            continue
387
                        process_node(prefix, node, path, self, self_pending)
388
                    if read_basis:
389
                        prefix, node, path = heapq.heappop(basis_pending)
390
                        if check_excluded(path):
391
                            continue
392
                        process_node(prefix, node, path, basis, basis_pending)
3735.2.32 by Robert Collins
Activate test for common node skipping. - 50 times performance improvement.
393
        # print loop_counter
3735.2.30 by Robert Collins
Start iter_changes between CHKMap instances.
394
3735.2.9 by Robert Collins
Get a working chk_map using inventory implementation bootstrapped.
395
    def iteritems(self, key_filter=None):
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
396
        """Iterate over the entire CHKMap's contents."""
397
        self._ensure_root()
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
398
        return self._root_node.iteritems(self._store, key_filter=key_filter)
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
399
3735.2.12 by Robert Collins
Implement commit-via-deltas for split inventory repositories.
400
    def key(self):
401
        """Return the key for this map."""
402
        if isinstance(self._root_node, tuple):
403
            return self._root_node
404
        else:
405
            return self._root_node._key
406
3735.2.17 by Robert Collins
Cache node length to avoid full iteration on __len__ calls.
407
    def __len__(self):
408
        self._ensure_root()
409
        return len(self._root_node)
410
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
411
    def map(self, key, value):
412
        """Map a key tuple to value."""
413
        # Need a root object.
414
        self._ensure_root()
415
        prefix, node_details = self._root_node.map(self._store, key, value)
416
        if len(node_details) == 1:
417
            self._root_node = node_details[0][1]
418
        else:
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
419
            self._root_node = InternalNode(prefix,
420
                                search_key_func=self._search_key_func)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
421
            self._root_node.set_maximum_size(node_details[0][1].maximum_size)
422
            self._root_node._key_width = node_details[0][1]._key_width
423
            for split, node in node_details:
424
                self._root_node.add_node(split, node)
425
3735.2.31 by Robert Collins
CHKMap.iter_changes
426
    def _node_key(self, node):
427
        """Get the key for a node whether its a tuple o r node."""
428
        if type(node) == tuple:
429
            return node
430
        else:
431
            return node._key
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
432
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
433
    def unmap(self, key):
434
        """remove key from the map."""
435
        self._ensure_root()
3735.11.3 by John Arbash Meinel
At the end of unmap() see if children can be packed into a single Leaf.
436
        unmapped = self._root_node.unmap(self._store, key)
437
        self._root_node = unmapped
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
438
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
439
    def _save(self):
440
        """Save the map completely.
441
442
        :return: The key of the root node.
443
        """
444
        if type(self._root_node) == tuple:
445
            # Already saved.
446
            return self._root_node
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
447
        keys = list(self._root_node.serialise(self._store))
448
        return keys[-1]
3735.2.8 by Robert Collins
New chk_map module for use in tree based inventory storage.
449
450
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
451
class Node(object):
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
452
    """Base class defining the protocol for CHK Map nodes.
453
454
    :ivar _raw_size: The total size of the serialized key:value data, before
455
        adding the header bytes, and without prefix compression.
456
    """
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
457
458
    def __init__(self, key_width=1):
459
        """Create a node.
460
461
        :param key_width: The width of keys for this node.
462
        """
463
        self._key = None
464
        # Current number of elements
465
        self._len = 0
466
        self._maximum_size = 0
467
        self._key_width = 1
468
        # current size in bytes
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
469
        self._raw_size = 0
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
470
        # The pointers/values this node has - meaning defined by child classes.
471
        self._items = {}
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
472
        # The common search prefix
473
        self._search_prefix = None
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
474
3735.9.4 by John Arbash Meinel
Some small cleanups, and fix _dump_tree to handle in-progress nodes.
475
    def __repr__(self):
476
        items_str = sorted(self._items)
477
        if len(items_str) > 20:
478
            items_str = items_str[16] + '...]'
3735.15.3 by John Arbash Meinel
repr update
479
        return '%s(key:%s len:%s size:%s max:%s prefix:%s items:%s)' % (
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
480
            self.__class__.__name__, self._key, self._len, self._raw_size,
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
481
            self._maximum_size, self._search_prefix, items_str)
3735.9.4 by John Arbash Meinel
Some small cleanups, and fix _dump_tree to handle in-progress nodes.
482
3735.2.38 by Robert Collins
Sufficient fixes to allow bzr-search to index a dev3 format repository.
483
    def key(self):
484
        return self._key
485
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
486
    def __len__(self):
487
        return self._len
488
489
    @property
490
    def maximum_size(self):
491
        """What is the upper limit for adding references to a node."""
492
        return self._maximum_size
493
494
    def set_maximum_size(self, new_size):
495
        """Set the size threshold for nodes.
496
497
        :param new_size: The size at which no data is added to a node. 0 for
498
            unlimited.
499
        """
500
        self._maximum_size = new_size
501
3735.15.5 by John Arbash Meinel
Change the nomenclature.
502
    @classmethod
503
    def common_prefix(cls, prefix, key):
504
        """Given 2 strings, return the longest prefix common to both.
505
506
        :param prefix: This has been the common prefix for other keys, so it is
507
            more likely to be the common prefix in this case as well.
508
        :param key: Another string to compare to
509
        """
510
        if key.startswith(prefix):
511
            return prefix
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
512
        # Is there a better way to do this?
3735.15.5 by John Arbash Meinel
Change the nomenclature.
513
        for pos, (left, right) in enumerate(zip(prefix, key)):
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
514
            if left != right:
3735.2.89 by Vincent Ladeuil
Fix the bogus previous fix.
515
                pos -= 1
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
516
                break
3735.2.89 by Vincent Ladeuil
Fix the bogus previous fix.
517
        common = prefix[:pos+1]
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
518
        return common
519
3735.15.5 by John Arbash Meinel
Change the nomenclature.
520
    @classmethod
521
    def common_prefix_for_keys(cls, keys):
522
        """Given a list of keys, find their common prefix.
523
524
        :param keys: An iterable of strings.
525
        :return: The longest common prefix of all keys.
526
        """
527
        common_prefix = None
528
        for key in keys:
529
            if common_prefix is None:
530
                common_prefix = key
531
                continue
532
            common_prefix = cls.common_prefix(common_prefix, key)
533
            if not common_prefix:
534
                # if common_prefix is the empty string, then we know it won't
535
                # change further
536
                return ''
537
        return common_prefix
538
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
539
540
class LeafNode(Node):
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
541
    """A node containing actual key:value pairs.
3735.11.1 by John Arbash Meinel
Clean up some trailing whitespace.
542
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
543
    :ivar _items: A dict of key->value items. The key is in tuple form.
3735.15.4 by John Arbash Meinel
Clean up some little bits.
544
    :ivar _size: The number of bytes that would be used by serializing all of
545
        the key/value pairs.
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
546
    """
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
547
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
548
    def __init__(self, search_key_func=None):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
549
        Node.__init__(self)
3735.15.4 by John Arbash Meinel
Clean up some little bits.
550
        # All of the keys in this leaf node share this common prefix
3735.15.5 by John Arbash Meinel
Change the nomenclature.
551
        self._common_serialised_prefix = None
552
        self._serialise_key = '\x00'.join
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
553
        if search_key_func is None:
3735.16.6 by John Arbash Meinel
Include a _search_key_plain function.
554
            self._search_key_func = _search_key_plain
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
555
        else:
556
            self._search_key_func = search_key_func
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
557
558
    def _current_size(self):
3735.15.4 by John Arbash Meinel
Clean up some little bits.
559
        """Answer the current serialised size of this node.
560
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
561
        This differs from self._raw_size in that it includes the bytes used for
562
        the header.
3735.15.4 by John Arbash Meinel
Clean up some little bits.
563
        """
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
564
        if self._common_serialised_prefix is None:
565
            bytes_for_items = 0
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
566
            prefix_len = 0
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
567
        else:
568
            # We will store a single string with the common prefix
569
            # And then that common prefix will not be stored in any of the
570
            # entry lines
571
            prefix_len = len(self._common_serialised_prefix)
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
572
            bytes_for_items = (self._raw_size - (prefix_len * self._len))
573
        return (9 # 'chkleaf:\n'
574
            + len(str(self._maximum_size)) + 1
575
            + len(str(self._key_width)) + 1
576
            + len(str(self._len)) + 1
577
            + prefix_len + 1
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
578
            + bytes_for_items)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
579
580
    @classmethod
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
581
    def deserialise(klass, bytes, key, search_key_func=None):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
582
        """Deserialise bytes, with key key, into a LeafNode.
583
584
        :param bytes: The bytes of the node.
585
        :param key: The key that the serialised node has.
586
        """
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
587
        result = LeafNode(search_key_func=search_key_func)
3735.2.72 by John Arbash Meinel
Change deserialise to properly handle when there is a '\r' in the key.
588
        # Splitlines can split on '\r' so don't use it, split('\n') adds an
589
        # extra '' if the bytes ends in a final newline.
590
        lines = bytes.split('\n')
591
        assert lines[-1] == ''
592
        lines.pop(-1)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
593
        items = {}
594
        if lines[0] != 'chkleaf:':
595
            raise ValueError("not a serialised leaf node: %r" % bytes)
596
        maximum_size = int(lines[1])
597
        width = int(lines[2])
598
        length = int(lines[3])
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
599
        prefix = lines[4]
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
600
        pos = 5
601
        while pos < len(lines):
602
            elements = (prefix + lines[pos]).split('\x00')
603
            pos += 1
604
            assert len(elements) == width + 1
605
            num_value_lines = int(elements[-1])
606
            value_lines = lines[pos:pos+num_value_lines]
607
            pos += num_value_lines
608
            value = '\n'.join(value_lines)
609
            items[tuple(elements[:-1])] = value
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
610
        if len(items) != length:
3735.14.3 by John Arbash Meinel
Properly remove keys that are found in the page cache. And add some debugging.
611
            raise AssertionError("item count (%d) mismatch for key %s,"
612
                " bytes %r" % (length, key, bytes))
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
613
        result._items = items
614
        result._len = length
615
        result._maximum_size = maximum_size
616
        result._key = key
617
        result._key_width = width
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
618
        result._raw_size = (sum(map(len, lines[5:])) # the length of the suffix
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
619
            + (length)*(len(prefix))
620
            + (len(lines)-5))
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
621
        result._compute_search_prefix()
3735.15.5 by John Arbash Meinel
Change the nomenclature.
622
        result._compute_serialised_prefix()
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
623
        if len(bytes) != result._current_size():
624
            import pdb; pdb.set_trace()
3735.15.8 by John Arbash Meinel
Add asserts so that when serializing and deserializing
625
        assert len(bytes) == result._current_size()
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
626
        return result
627
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
628
    def iteritems(self, store, key_filter=None):
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
629
        """Iterate over items in the node.
630
631
        :param key_filter: A filter to apply to the node. It should be a
632
            list/set/dict or similar repeatedly iterable container.
633
        """
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
634
        if key_filter is not None:
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
635
            # Adjust the filter - short elements go to a prefix filter. Would this
636
            # be cleaner explicitly? That would be no harder for InternalNode..
637
            # XXX: perhaps defaultdict? Profiling<rinse and repeat>
638
            filters = {}
639
            for key in key_filter:
640
                length_filter = filters.setdefault(len(key), set())
641
                length_filter.add(key)
642
            filters = filters.items()
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
643
            for item in self._items.iteritems():
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
644
                for length, length_filter in filters:
645
                    if item[0][:length] in length_filter:
646
                        yield item
647
                        break
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
648
        else:
649
            for item in self._items.iteritems():
650
                yield item
651
3735.9.4 by John Arbash Meinel
Some small cleanups, and fix _dump_tree to handle in-progress nodes.
652
    def _key_value_len(self, key, value):
653
        # TODO: Should probably be done without actually joining the key, but
654
        #       then that can be done via the C extension
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
655
        return (len(self._serialise_key(key)) + 1
656
                + len(str(value.count('\n'))) + 1
657
                + len(value) + 1)
3735.9.4 by John Arbash Meinel
Some small cleanups, and fix _dump_tree to handle in-progress nodes.
658
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
659
    def _search_key(self, key):
660
        return self._search_key_func(key)
661
3735.11.13 by John Arbash Meinel
Refactor the LeafNode.map() code so we can do _check_remap more cheaply.
662
    def _map_no_split(self, key, value):
663
        """Map a key to a value.
664
665
        This assumes either the key does not already exist, or you have already
666
        removed its size and length from self.
667
668
        :return: True if adding this node should cause us to split.
669
        """
670
        self._items[key] = value
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
671
        self._raw_size += self._key_value_len(key, value)
3735.11.13 by John Arbash Meinel
Refactor the LeafNode.map() code so we can do _check_remap more cheaply.
672
        self._len += 1
3735.15.5 by John Arbash Meinel
Change the nomenclature.
673
        serialised_key = self._serialise_key(key)
674
        if self._common_serialised_prefix is None:
675
            self._common_serialised_prefix = serialised_key
676
        else:
677
            self._common_serialised_prefix = self.common_prefix(
678
                self._common_serialised_prefix, serialised_key)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
679
        search_key = self._search_key(key)
680
        if self._search_prefix is None:
681
            self._search_prefix = search_key
3735.15.5 by John Arbash Meinel
Change the nomenclature.
682
        else:
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
683
            self._search_prefix = self.common_prefix(
684
                self._search_prefix, search_key)
3735.11.13 by John Arbash Meinel
Refactor the LeafNode.map() code so we can do _check_remap more cheaply.
685
        if (self._len > 1
686
            and self._maximum_size
3735.16.10 by John Arbash Meinel
Don't track state for an infrequent edge case.
687
            and self._current_size() > self._maximum_size):
688
            # Check to see if all of the search_keys for this node are
689
            # identical. We allow the node to grow under that circumstance
690
            # (we could track this as common state, but it is infrequent)
691
            if (search_key != self._search_prefix
692
                or not self._are_search_keys_identical()):
693
                return True
3735.11.13 by John Arbash Meinel
Refactor the LeafNode.map() code so we can do _check_remap more cheaply.
694
        return False
695
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
696
    def _split(self, store):
697
        """We have overflowed.
698
699
        Split this node into multiple LeafNodes, return it up the stack so that
700
        the next layer creates a new InternalNode and references the new nodes.
701
3735.15.5 by John Arbash Meinel
Change the nomenclature.
702
        :return: (common_serialised_prefix, [(node_serialised_prefix, node)])
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
703
        """
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
704
        common_prefix = self._search_prefix
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
705
        split_at = len(common_prefix) + 1
706
        result = {}
707
        for key, value in self._items.iteritems():
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
708
            search_key = self._search_key(key)
709
            prefix = search_key[:split_at]
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
710
            # TODO: Generally only 1 key can be exactly the right length,
711
            #       which means we can only have 1 key in the node pointed
712
            #       at by the 'prefix\0' key. We might want to consider
713
            #       folding it into the containing InternalNode rather than
714
            #       having a fixed length-1 node.
715
            #       Note this is probably not true for hash keys, as they
716
            #       may get a '\00' node anywhere, but won't have keys of
717
            #       different lengths.
718
            if len(prefix) < split_at:
719
                prefix += '\x00'*(split_at - len(prefix))
720
            if prefix not in result:
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
721
                node = LeafNode(search_key_func=self._search_key_func)
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
722
                node.set_maximum_size(self._maximum_size)
723
                node._key_width = self._key_width
724
                result[prefix] = node
725
            else:
726
                node = result[prefix]
727
            node.map(store, key, value)
728
        return common_prefix, result.items()
729
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
730
    def map(self, store, key, value):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
731
        """Map key to value."""
732
        if key in self._items:
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
733
            self._raw_size -= self._key_value_len(key, self._items[key])
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
734
            self._len -= 1
735
        self._key = None
3735.11.13 by John Arbash Meinel
Refactor the LeafNode.map() code so we can do _check_remap more cheaply.
736
        if self._map_no_split(key, value):
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
737
            return self._split(store)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
738
        else:
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
739
            return self._search_prefix, [("", self)]
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
740
741
    def serialise(self, store):
742
        """Serialise the tree to store.
743
744
        :param store: A VersionedFiles honouring the CHK extensions.
745
        :return: An iterable of the keys inserted by this operation.
746
        """
747
        lines = ["chkleaf:\n"]
748
        lines.append("%d\n" % self._maximum_size)
749
        lines.append("%d\n" % self._key_width)
750
        lines.append("%d\n" % self._len)
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
751
        if self._common_serialised_prefix is None:
752
            lines.append('\n')
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
753
            assert len(self._items) == 0
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
754
        else:
755
            lines.append('%s\n' % (self._common_serialised_prefix,))
756
            prefix_len = len(self._common_serialised_prefix)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
757
        for key, value in sorted(self._items.items()):
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
758
            # Add always add a final newline
759
            value_lines = osutils.chunks_to_lines([value + '\n'])
760
            serialized = "%s\x00%s\n" % (self._serialise_key(key),
761
                                         len(value_lines))
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
762
            assert serialized.startswith(self._common_serialised_prefix)
763
            lines.append(serialized[prefix_len:])
3735.17.1 by John Arbash Meinel
Change the serialized form for leaf nodes.
764
            lines.extend(value_lines)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
765
        sha1, _, _ = store.add_lines((None,), (), lines)
766
        self._key = ("sha1:" + sha1,)
3735.15.8 by John Arbash Meinel
Add asserts so that when serializing and deserializing
767
        bytes = ''.join(lines)
3735.15.9 by John Arbash Meinel
(broken) Initial prototype of leaf pages which pull out their common prefix.
768
        if len(bytes) != self._current_size():
769
            import pdb; pdb.set_trace()
3735.15.8 by John Arbash Meinel
Add asserts so that when serializing and deserializing
770
        assert len(bytes) == self._current_size()
771
        _page_cache.add(self._key, bytes)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
772
        return [self._key]
773
3735.2.26 by Robert Collins
CHKInventory migrated to new CHKMap code.
774
    def refs(self):
775
        """Return the references to other CHK's held by this node."""
776
        return []
777
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
778
    def _compute_search_prefix(self):
779
        """Determine the common search prefix for all keys in this node.
3735.15.5 by John Arbash Meinel
Change the nomenclature.
780
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
781
        :return: A bytestring of the longest search key prefix that is
3735.15.5 by John Arbash Meinel
Change the nomenclature.
782
            unique within this node.
783
        """
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
784
        search_keys = [self._search_key(key) for key in self._items]
785
        self._search_prefix = self.common_prefix_for_keys(search_keys)
786
        return self._search_prefix
3735.15.5 by John Arbash Meinel
Change the nomenclature.
787
3735.16.10 by John Arbash Meinel
Don't track state for an infrequent edge case.
788
    def _are_search_keys_identical(self):
789
        """Check to see if the search keys for all entries are the same.
790
791
        When using a hash as the search_key it is possible for non-identical
792
        keys to collide. If that happens enough, we may try overflow a
793
        LeafNode, but as all are collisions, we must not split.
794
        """
795
        common_search_key = None
796
        for key in self._items:
797
            search_key = self._search_key(key)
798
            if common_search_key is None:
799
                common_search_key = search_key
800
            elif search_key != common_search_key:
801
                return False
802
        return True
803
3735.15.5 by John Arbash Meinel
Change the nomenclature.
804
    def _compute_serialised_prefix(self):
805
        """Determine the common prefix for serialised keys in this node.
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
806
807
        :return: A bytestring of the longest serialised key prefix that is
808
            unique within this node.
809
        """
3735.15.5 by John Arbash Meinel
Change the nomenclature.
810
        serialised_keys = [self._serialise_key(key) for key in self._items]
811
        self._common_serialised_prefix = self.common_prefix_for_keys(
812
            serialised_keys)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
813
814
    def unmap(self, store, key):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
815
        """Unmap key from the node."""
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
816
        self._raw_size -= self._key_value_len(key, self._items[key])
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
817
        self._len -= 1
818
        del self._items[key]
819
        self._key = None
3735.15.2 by John Arbash Meinel
Change LeafNode to also cache its unique serialized prefix.
820
        # Recompute from scratch
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
821
        self._compute_search_prefix()
3735.15.5 by John Arbash Meinel
Change the nomenclature.
822
        self._compute_serialised_prefix()
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
823
        return self
824
825
826
class InternalNode(Node):
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
827
    """A node that contains references to other nodes.
3735.11.1 by John Arbash Meinel
Clean up some trailing whitespace.
828
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
829
    An InternalNode is responsible for mapping search key prefixes to child
3735.15.5 by John Arbash Meinel
Change the nomenclature.
830
    nodes.
3735.15.4 by John Arbash Meinel
Clean up some little bits.
831
3735.15.5 by John Arbash Meinel
Change the nomenclature.
832
    :ivar _items: serialised_key => node dictionary. node may be a tuple,
3735.15.4 by John Arbash Meinel
Clean up some little bits.
833
        LeafNode or InternalNode.
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
834
    """
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
835
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
836
    def __init__(self, prefix='', search_key_func=None):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
837
        Node.__init__(self)
838
        # The size of an internalnode with default values and no children.
839
        # How many octets key prefixes within this node are.
840
        self._node_width = 0
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
841
        self._search_prefix = prefix
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
842
        if search_key_func is None:
3735.16.6 by John Arbash Meinel
Include a _search_key_plain function.
843
            self._search_key_func = _search_key_plain
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
844
        else:
845
            self._search_key_func = search_key_func
3735.9.5 by John Arbash Meinel
Don't allow an InternalNode to add a key that doesn't fit.
846
847
    def __repr__(self):
848
        items_str = sorted(self._items)
849
        if len(items_str) > 20:
850
            items_str = items_str[16] + '...]'
851
        return '%s(key:%s len:%s size:%s max:%s prefix:%s items:%s)' % (
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
852
            self.__class__.__name__, self._key, self._len, self._raw_size,
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
853
            self._maximum_size, self._search_prefix, items_str)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
854
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
855
    def add_node(self, prefix, node):
856
        """Add a child node with prefix prefix, and node node.
857
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
858
        :param prefix: The search key prefix for node.
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
859
        :param node: The node being added.
860
        """
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
861
        assert self._search_prefix is not None
862
        assert prefix.startswith(self._search_prefix)
863
        assert len(prefix) == len(self._search_prefix) + 1
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
864
        self._len += len(node)
865
        if not len(self._items):
866
            self._node_width = len(prefix)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
867
        assert self._node_width == len(self._search_prefix) + 1
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
868
        self._items[prefix] = node
869
        self._key = None
870
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
871
    def _current_size(self):
872
        """Answer the current serialised size of this node."""
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
873
        return (self._raw_size + len(str(self._len)) + len(str(self._key_width)) +
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
874
            len(str(self._maximum_size)))
875
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
876
    @classmethod
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
877
    def deserialise(klass, bytes, key, search_key_func=None):
3735.2.25 by Robert Collins
CHKInventory core tests passing.
878
        """Deserialise bytes to an InternalNode, with key key.
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
879
880
        :param bytes: The bytes of the node.
881
        :param key: The key that the serialised node has.
882
        :return: An InternalNode instance.
883
        """
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
884
        result = InternalNode(search_key_func=search_key_func)
3735.2.72 by John Arbash Meinel
Change deserialise to properly handle when there is a '\r' in the key.
885
        # Splitlines can split on '\r' so don't use it, remove the extra ''
886
        # from the result of split('\n') because we should have a trailing
887
        # newline
888
        lines = bytes.split('\n')
889
        assert lines[-1] == ''
890
        lines.pop(-1)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
891
        items = {}
892
        if lines[0] != 'chknode:':
893
            raise ValueError("not a serialised internal node: %r" % bytes)
894
        maximum_size = int(lines[1])
895
        width = int(lines[2])
896
        length = int(lines[3])
3735.15.11 by John Arbash Meinel
Change the InternalNodes to also pull out the common prefix.
897
        common_prefix = lines[4]
898
        for line in lines[5:]:
899
            line = common_prefix + line
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
900
            prefix, flat_key = line.rsplit('\x00', 1)
901
            items[prefix] = (flat_key,)
902
        result._items = items
903
        result._len = length
904
        result._maximum_size = maximum_size
905
        result._key = key
906
        result._key_width = width
3735.15.6 by John Arbash Meinel
Add tests that LeafNodes track the common prefix for both their lookup keys
907
        # XXX: InternalNodes don't really care about their size, and this will
908
        #      change if we add prefix compression
909
        result._raw_size = None # len(bytes)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
910
        result._node_width = len(prefix)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
911
        result._compute_search_prefix()
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
912
        return result
913
914
    def iteritems(self, store, key_filter=None):
915
        for node in self._iter_nodes(store, key_filter=key_filter):
916
            for item in node.iteritems(store, key_filter=key_filter):
917
                yield item
918
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
919
    def _iter_nodes(self, store, key_filter=None, batch_size=None):
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
920
        """Iterate over node objects which match key_filter.
921
922
        :param store: A store to use for accessing content.
923
        :param key_filter: A key filter to filter nodes. Only nodes that might
924
            contain a key in key_filter will be returned.
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
925
        :param batch_size: If not None, then we will return the nodes that had
926
            to be read using get_record_stream in batches, rather than reading
927
            them all at once.
928
        :return: An iterable of nodes. This function does not have to be fully
929
            consumed.  (There will be no pending I/O when items are being returned.)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
930
        """
3735.2.31 by Robert Collins
CHKMap.iter_changes
931
        keys = {}
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
932
        if key_filter is None:
3735.2.31 by Robert Collins
CHKMap.iter_changes
933
            for prefix, node in self._items.iteritems():
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
934
                if type(node) == tuple:
3735.2.31 by Robert Collins
CHKMap.iter_changes
935
                    keys[node] = prefix
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
936
                else:
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
937
                    yield node
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
938
        else:
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
939
            # XXX defaultdict ?
940
            length_filters = {}
941
            for key in key_filter:
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
942
                search_key = self._search_prefix_filter(key)
943
                length_filter = length_filters.setdefault(
944
                                    len(search_key), set())
945
                length_filter.add(search_key)
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
946
            length_filters = length_filters.items()
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
947
            for prefix, node in self._items.iteritems():
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
948
                for length, length_filter in length_filters:
949
                    if prefix[:length] in length_filter:
950
                        if type(node) == tuple:
951
                            keys[node] = prefix
952
                        else:
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
953
                            yield node
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
954
                        break
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
955
        if keys:
3735.2.62 by Robert Collins
Create a rudimentary CHK page cache.
956
            # Look in the page cache for some more bytes
957
            found_keys = set()
958
            for key in keys:
959
                try:
960
                    bytes = _page_cache[key]
961
                except KeyError:
962
                    continue
963
                else:
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
964
                    node = _deserialise(bytes, key,
965
                        search_key_func=self._search_key_func)
3735.2.62 by Robert Collins
Create a rudimentary CHK page cache.
966
                    self._items[keys[key]] = node
967
                    found_keys.add(key)
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
968
                    yield node
3735.2.62 by Robert Collins
Create a rudimentary CHK page cache.
969
            for key in found_keys:
970
                del keys[key]
971
        if keys:
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
972
            # demand load some pages.
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
973
            if batch_size is None:
974
                # Read all the keys in
975
                batch_size = len(keys)
976
            key_order = list(keys)
977
            for batch_start in range(0, len(key_order), batch_size):
978
                batch = key_order[batch_start:batch_start + batch_size]
979
                # We have to fully consume the stream so there is no pending
980
                # I/O, so we buffer the nodes for now.
981
                stream = store.get_record_stream(batch, 'unordered', True)
982
                nodes = []
983
                for record in stream:
984
                    bytes = record.get_bytes_as('fulltext')
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
985
                    node = _deserialise(bytes, record.key,
986
                        search_key_func=self._search_key_func)
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
987
                    nodes.append(node)
988
                    self._items[keys[record.key]] = node
989
                    _page_cache.add(record.key, bytes)
990
                for node in nodes:
991
                    yield node
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
992
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
993
    def map(self, store, key, value):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
994
        """Map key to value."""
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
995
        if not len(self._items):
996
            raise AssertionError("cant map in an empty InternalNode.")
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
997
        search_key = self._search_key(key)
998
        assert self._node_width == len(self._search_prefix) + 1
999
        if not search_key.startswith(self._search_prefix):
3735.9.11 by John Arbash Meinel
Handle when an InternalNode decides it needs to split.
1000
            # This key doesn't fit in this index, so we need to split at the
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
1001
            # point where it would fit, insert self into that internal node,
1002
            # and then map this key into that node.
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1003
            new_prefix = self.common_prefix(self._search_prefix,
1004
                                            search_key)
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1005
            new_parent = InternalNode(new_prefix,
1006
                search_key_func=self._search_key_func)
3735.9.5 by John Arbash Meinel
Don't allow an InternalNode to add a key that doesn't fit.
1007
            new_parent.set_maximum_size(self._maximum_size)
1008
            new_parent._key_width = self._key_width
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1009
            new_parent.add_node(self._search_prefix[:len(new_prefix)+1],
3735.15.1 by John Arbash Meinel
Change InternalNode to always cache its serialized_prefix.
1010
                                self)
3735.9.5 by John Arbash Meinel
Don't allow an InternalNode to add a key that doesn't fit.
1011
            return new_parent.map(store, key, value)
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
1012
        children = list(self._iter_nodes(store, key_filter=[key]))
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1013
        if children:
1014
            child = children[0]
1015
        else:
1016
            # new child needed:
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1017
            child = self._new_child(search_key, LeafNode)
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
1018
        old_len = len(child)
3735.11.11 by John Arbash Meinel
Add logic to map() so that it can also collapse when necessary.
1019
        if isinstance(child, LeafNode):
1020
            old_size = child._current_size()
1021
        else:
1022
            old_size = None
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1023
        prefix, node_details = child.map(store, key, value)
1024
        if len(node_details) == 1:
3735.9.11 by John Arbash Meinel
Handle when an InternalNode decides it needs to split.
1025
            # child may have shrunk, or might be a new node
1026
            child = node_details[0][1]
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1027
            self._len = self._len - old_len + len(child)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1028
            self._items[search_key] = child
3735.2.29 by Robert Collins
Untested code is broken code.
1029
            self._key = None
3735.11.11 by John Arbash Meinel
Add logic to map() so that it can also collapse when necessary.
1030
            new_node = self
1031
            if (isinstance(child, LeafNode)
1032
                and (old_size is None or child._current_size() < old_size)):
1033
                # The old node was an InternalNode which means it has now
1034
                # collapsed, so we need to check if it will chain to a collapse
1035
                # at this level. Or the LeafNode has shrunk in size, so we need
1036
                # to check that as well.
1037
                new_node = self._check_remap(store)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1038
            assert new_node._search_prefix is not None
1039
            return new_node._search_prefix, [('', new_node)]
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1040
        # child has overflown - create a new intermediate node.
1041
        # XXX: This is where we might want to try and expand our depth
1042
        # to refer to more bytes of every child (which would give us
1043
        # multiple pointers to child nodes, but less intermediate nodes)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1044
        child = self._new_child(search_key, InternalNode)
1045
        child._search_prefix = prefix
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1046
        for split, node in node_details:
1047
            child.add_node(split, node)
1048
        self._len = self._len - old_len + len(child)
3735.2.29 by Robert Collins
Untested code is broken code.
1049
        self._key = None
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1050
        return self._search_prefix, [("", self)]
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1051
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1052
    def _new_child(self, search_key, klass):
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1053
        """Create a new child node of type klass."""
1054
        child = klass()
1055
        child.set_maximum_size(self._maximum_size)
1056
        child._key_width = self._key_width
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1057
        child._search_key_func = self._search_key_func
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1058
        self._items[search_key] = child
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1059
        return child
1060
1061
    def serialise(self, store):
1062
        """Serialise the node to store.
1063
1064
        :param store: A VersionedFiles honouring the CHK extensions.
1065
        :return: An iterable of the keys inserted by this operation.
1066
        """
1067
        for node in self._items.itervalues():
1068
            if type(node) == tuple:
1069
                # Never deserialised.
1070
                continue
1071
            if node._key is not None:
1072
                # Never altered
1073
                continue
1074
            for key in node.serialise(store):
1075
                yield key
1076
        lines = ["chknode:\n"]
1077
        lines.append("%d\n" % self._maximum_size)
1078
        lines.append("%d\n" % self._key_width)
1079
        lines.append("%d\n" % self._len)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1080
        assert self._search_prefix is not None
1081
        lines.append('%s\n' % (self._search_prefix,))
1082
        prefix_len = len(self._search_prefix)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1083
        for prefix, node in sorted(self._items.items()):
1084
            if type(node) == tuple:
1085
                key = node[0]
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
1086
            else:
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1087
                key = node._key[0]
3735.15.11 by John Arbash Meinel
Change the InternalNodes to also pull out the common prefix.
1088
            serialised = "%s\x00%s\n" % (prefix, key)
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1089
            assert serialised.startswith(self._search_prefix)
3735.15.11 by John Arbash Meinel
Change the InternalNodes to also pull out the common prefix.
1090
            lines.append(serialised[prefix_len:])
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1091
        sha1, _, _ = store.add_lines((None,), (), lines)
1092
        self._key = ("sha1:" + sha1,)
3735.2.63 by Robert Collins
Divert writes into the CHK page cache as well.
1093
        _page_cache.add(self._key, ''.join(lines))
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1094
        yield self._key
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
1095
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1096
    def _search_key(self, key):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
1097
        """Return the serialised key for key in this node."""
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1098
        # search keys are fixed width. All will be self._node_width wide, so we
3735.15.5 by John Arbash Meinel
Change the nomenclature.
1099
        # pad as necessary.
3735.16.1 by John Arbash Meinel
(broken) Start tracking down more code that needs to pass around the 'search_key_func'
1100
        return (self._search_key_func(key) + '\x00'*self._node_width)[:self._node_width]
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
1101
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1102
    def _search_prefix_filter(self, key):
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
1103
        """Serialise key for use as a prefix filter in iteritems."""
1104
        if len(key) == self._key_width:
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1105
            return self._search_key(key)
3735.2.43 by Robert Collins
Teach CHKMap how to iter items in 2-tuple keyspaces.
1106
        return '\x00'.join(key)[:self._node_width]
1107
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1108
    def _split(self, offset):
1109
        """Split this node into smaller nodes starting at offset.
1110
1111
        :param offset: The offset to start the new child nodes at.
1112
        :return: An iterable of (prefix, node) tuples. prefix is a byte
1113
            prefix for reaching node.
1114
        """
1115
        if offset >= self._node_width:
1116
            for node in self._items.values():
1117
                for result in node._split(offset):
1118
                    yield result
1119
            return
1120
        for key, node in self._items.items():
1121
            pass
1122
3735.2.26 by Robert Collins
CHKInventory migrated to new CHKMap code.
1123
    def refs(self):
1124
        """Return the references to other CHK's held by this node."""
1125
        if self._key is None:
1126
            raise AssertionError("unserialised nodes have no refs.")
1127
        refs = []
1128
        for value in self._items.itervalues():
1129
            if type(value) == tuple:
1130
                refs.append(value)
1131
            else:
1132
                refs.append(value.key())
1133
        return refs
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1134
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1135
    def _compute_search_prefix(self, extra_key=None):
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1136
        """Return the unique key prefix for this node.
1137
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1138
        :return: A bytestring of the longest search key prefix that is
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1139
            unique within this node.
1140
        """
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1141
        self._search_prefix = self.common_prefix_for_keys(self._items)
1142
        return self._search_prefix
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1143
1144
    def unmap(self, store, key):
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
1145
        """Remove key from this node and it's children."""
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1146
        if not len(self._items):
1147
            raise AssertionError("cant unmap in an empty InternalNode.")
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
1148
        children = list(self._iter_nodes(store, key_filter=[key]))
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1149
        if children:
1150
            child = children[0]
1151
        else:
1152
            raise KeyError(key)
1153
        self._len -= 1
1154
        unmapped = child.unmap(store, key)
3735.11.3 by John Arbash Meinel
At the end of unmap() see if children can be packed into a single Leaf.
1155
        self._key = None
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1156
        search_key = self._search_key(key)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1157
        if len(unmapped) == 0:
1158
            # All child nodes are gone, remove the child:
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1159
            del self._items[search_key]
3735.11.3 by John Arbash Meinel
At the end of unmap() see if children can be packed into a single Leaf.
1160
            unmapped = None
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1161
        else:
1162
            # Stash the returned node
3735.15.13 by John Arbash Meinel
Change the term 'lookup' to the term 'search', as it is closer to what Robert envisioned.
1163
            self._items[search_key] = unmapped
3735.2.23 by Robert Collins
Test unmapping with one child left but multiple keys.
1164
        if len(self._items) == 1:
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1165
            # this node is no longer needed:
1166
            return self._items.values()[0]
3735.11.10 by John Arbash Meinel
Change how _check_remap works so it doesn't have to load all keys.
1167
        if isinstance(unmapped, InternalNode):
1168
            return self
1169
        return self._check_remap(store)
1170
1171
    def _check_remap(self, store):
3735.11.11 by John Arbash Meinel
Add logic to map() so that it can also collapse when necessary.
1172
        """Check if all keys contained by children fit in a single LeafNode.
1173
1174
        :param store: A store to use for reading more nodes
1175
        :return: Either self, or a new LeafNode which should replace self.
1176
        """
3735.11.10 by John Arbash Meinel
Change how _check_remap works so it doesn't have to load all keys.
1177
        # Logic for how we determine when we need to rebuild
3735.11.3 by John Arbash Meinel
At the end of unmap() see if children can be packed into a single Leaf.
1178
        # 1) Implicitly unmap() is removing a key which means that the child
1179
        #    nodes are going to be shrinking by some extent.
1180
        # 2) If all children are LeafNodes, it is possible that they could be
1181
        #    combined into a single LeafNode, which can then completely replace
1182
        #    this internal node with a single LeafNode
1183
        # 3) If *one* child is an InternalNode, we assume it has already done
1184
        #    all the work to determine that its children cannot collapse, and
1185
        #    we can then assume that those nodes *plus* the current nodes don't
1186
        #    have a chance of collapsing either.
1187
        #    So a very cheap check is to just say if 'unmapped' is an
1188
        #    InternalNode, we don't have to check further.
3735.11.10 by John Arbash Meinel
Change how _check_remap works so it doesn't have to load all keys.
1189
3735.11.3 by John Arbash Meinel
At the end of unmap() see if children can be packed into a single Leaf.
1190
        # TODO: Another alternative is to check the total size of all known
1191
        #       LeafNodes. If there is some formula we can use to determine the
1192
        #       final size without actually having to read in any more
1193
        #       children, it would be nice to have. However, we have to be
1194
        #       careful with stuff like nodes that pull out the common prefix
1195
        #       of each key, as adding a new key can change the common prefix
1196
        #       and cause size changes greater than the length of one key.
1197
        #       So for now, we just add everything to a new Leaf until it
1198
        #       splits, as we know that will give the right answer
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1199
        new_leaf = LeafNode(search_key_func=self._search_key_func)
3735.11.3 by John Arbash Meinel
At the end of unmap() see if children can be packed into a single Leaf.
1200
        new_leaf.set_maximum_size(self._maximum_size)
1201
        new_leaf._key_width = self._key_width
3735.14.7 by John Arbash Meinel
Change _iter_nodes into a generator.
1202
        # A batch_size of 16 was chosen because:
1203
        #   a) In testing, a 4k page held 14 times. So if we have more than 16
1204
        #      leaf nodes we are unlikely to hold them in a single new leaf
1205
        #      node. This still allows for 1 round trip
1206
        #   b) With 16-way fan out, we can still do a single round trip
1207
        #   c) With 255-way fan out, we don't want to read all 255 and destroy
1208
        #      the page cache, just to determine that we really don't need it.
1209
        for node in self._iter_nodes(store, batch_size=16):
1210
            if isinstance(node, InternalNode):
1211
                # Without looking at any leaf nodes, we are sure
1212
                return self
1213
            for key, value in node._items.iteritems():
1214
                if new_leaf._map_no_split(key, value):
3735.11.10 by John Arbash Meinel
Change how _check_remap works so it doesn't have to load all keys.
1215
                    return self
3735.11.3 by John Arbash Meinel
At the end of unmap() see if children can be packed into a single Leaf.
1216
        return new_leaf
3735.2.18 by Robert Collins
Partial multi-layer chk dictionary trees.
1217
1218
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1219
def _deserialise(bytes, key, search_key_func):
3735.2.16 by Robert Collins
Untested extensions to support repodetails
1220
    """Helper for repositorydetails - convert bytes to a node."""
3735.2.24 by Robert Collins
test_chk_map tests all passing.
1221
    if bytes.startswith("chkleaf:\n"):
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1222
        return LeafNode.deserialise(bytes, key, search_key_func=search_key_func)
3735.2.21 by Robert Collins
BROKEN: multi level CHKMap tries, unfinished, subsystem in flux.
1223
    elif bytes.startswith("chknode:\n"):
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1224
        return InternalNode.deserialise(bytes, key,
1225
            search_key_func=search_key_func)
3735.2.16 by Robert Collins
Untested extensions to support repodetails
1226
    else:
1227
        raise AssertionError("Unknown node type.")
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1228
1229
3735.2.67 by John Arbash Meinel
Merge bzr.dev 3903 which brings in 'chunked' encoding.
1230
def _find_children_info(store, interesting_keys, uninteresting_keys, pb):
3735.9.7 by John Arbash Meinel
Cleanup pass.
1231
    """Read the associated records, and determine what is interesting."""
1232
    uninteresting_keys = set(uninteresting_keys)
1233
    chks_to_read = uninteresting_keys.union(interesting_keys)
1234
    next_uninteresting = set()
1235
    next_interesting = set()
1236
    uninteresting_items = set()
1237
    interesting_items = set()
1238
    interesting_records = []
3735.9.14 by John Arbash Meinel
Start using the iter_interesting_nodes.
1239
    # records_read = set()
3735.9.7 by John Arbash Meinel
Cleanup pass.
1240
    for record in store.get_record_stream(chks_to_read, 'unordered', True):
3735.9.14 by John Arbash Meinel
Start using the iter_interesting_nodes.
1241
        # records_read.add(record.key())
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1242
        if pb is not None:
1243
            pb.tick()
3735.2.67 by John Arbash Meinel
Merge bzr.dev 3903 which brings in 'chunked' encoding.
1244
        bytes = record.get_bytes_as('fulltext')
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1245
        # We don't care about search_key_func for this code, because we only
1246
        # care about external references.
1247
        node = _deserialise(bytes, record.key, search_key_func=None)
3735.9.7 by John Arbash Meinel
Cleanup pass.
1248
        if record.key in uninteresting_keys:
1249
            if isinstance(node, InternalNode):
3735.9.14 by John Arbash Meinel
Start using the iter_interesting_nodes.
1250
                next_uninteresting.update(node.refs())
3735.9.7 by John Arbash Meinel
Cleanup pass.
1251
            else:
3735.9.14 by John Arbash Meinel
Start using the iter_interesting_nodes.
1252
                # We know we are at a LeafNode, so we can pass None for the
1253
                # store
1254
                uninteresting_items.update(node.iteritems(None))
3735.9.7 by John Arbash Meinel
Cleanup pass.
1255
        else:
1256
            interesting_records.append(record)
1257
            if isinstance(node, InternalNode):
3735.9.14 by John Arbash Meinel
Start using the iter_interesting_nodes.
1258
                next_interesting.update(node.refs())
3735.9.7 by John Arbash Meinel
Cleanup pass.
1259
            else:
3735.9.14 by John Arbash Meinel
Start using the iter_interesting_nodes.
1260
                interesting_items.update(node.iteritems(None))
1261
    # TODO: Filter out records that have already been read, as node splitting
1262
    #       can cause us to reference the same nodes via shorter and longer
1263
    #       paths
3735.9.7 by John Arbash Meinel
Cleanup pass.
1264
    return (next_uninteresting, uninteresting_items,
1265
            next_interesting, interesting_records, interesting_items)
1266
1267
3735.9.19 by John Arbash Meinel
Refactor iter_interesting a little bit.
1268
def _find_all_uninteresting(store, interesting_root_keys,
1269
                            uninteresting_root_keys, adapter, pb):
1270
    """Determine the full set of uninteresting keys."""
1271
    # What about duplicates between interesting_root_keys and
1272
    # uninteresting_root_keys?
1273
    if not uninteresting_root_keys:
1274
        # Shortcut case. We know there is nothing uninteresting to filter out
1275
        # So we just let the rest of the algorithm do the work
1276
        # We know there is nothing uninteresting, and we didn't have to read
1277
        # any interesting records yet.
1278
        return (set(), set(), set(interesting_root_keys), [], set())
3735.9.7 by John Arbash Meinel
Cleanup pass.
1279
    all_uninteresting_chks = set(uninteresting_root_keys)
1280
    all_uninteresting_items = set()
1281
1282
    # First step, find the direct children of both the interesting and
1283
    # uninteresting set
1284
    (uninteresting_keys, uninteresting_items,
1285
     interesting_keys, interesting_records,
1286
     interesting_items) = _find_children_info(store, interesting_root_keys,
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1287
                                              uninteresting_root_keys,
3735.2.67 by John Arbash Meinel
Merge bzr.dev 3903 which brings in 'chunked' encoding.
1288
                                              pb=pb)
3735.9.7 by John Arbash Meinel
Cleanup pass.
1289
    all_uninteresting_chks.update(uninteresting_keys)
1290
    all_uninteresting_items.update(uninteresting_items)
1291
    del uninteresting_items
1292
    # Note: Exact matches between interesting and uninteresting do not need
1293
    #       to be search further. Non-exact matches need to be searched in case
1294
    #       there is a future exact-match
1295
    uninteresting_keys.difference_update(interesting_keys)
1296
3735.9.6 by John Arbash Meinel
Add a first pass to the interesting search.
1297
    # Second, find the full set of uninteresting bits reachable by the
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1298
    # uninteresting roots
3735.9.7 by John Arbash Meinel
Cleanup pass.
1299
    chks_to_read = uninteresting_keys
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1300
    while chks_to_read:
1301
        next_chks = set()
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1302
        for record in store.get_record_stream(chks_to_read, 'unordered', False):
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1303
            # TODO: Handle 'absent'
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1304
            if pb is not None:
1305
                pb.tick()
3735.2.67 by John Arbash Meinel
Merge bzr.dev 3903 which brings in 'chunked' encoding.
1306
            if record.storage_kind in ('fulltext', 'chunked'):
1307
                bytes = record.get_bytes_as('fulltext')
1308
            else:
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1309
                bytes = adapter.get_bytes(record,
1310
                            record.get_bytes_as(record.storage_kind))
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1311
            # We don't care about search_key_func for this code, because we
1312
            # only care about external references.
1313
            node = _deserialise(bytes, record.key, search_key_func=None)
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1314
            if isinstance(node, InternalNode):
1315
                # uninteresting_prefix_chks.update(node._items.iteritems())
1316
                chks = node._items.values()
1317
                # TODO: We remove the entries that are already in
1318
                #       uninteresting_chks ?
1319
                next_chks.update(chks)
3735.9.7 by John Arbash Meinel
Cleanup pass.
1320
                all_uninteresting_chks.update(chks)
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1321
            else:
3735.9.7 by John Arbash Meinel
Cleanup pass.
1322
                all_uninteresting_items.update(node._items.iteritems())
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1323
        chks_to_read = next_chks
3735.9.19 by John Arbash Meinel
Refactor iter_interesting a little bit.
1324
    return (all_uninteresting_chks, all_uninteresting_items,
1325
            interesting_keys, interesting_records, interesting_items)
1326
1327
1328
def iter_interesting_nodes(store, interesting_root_keys,
1329
                           uninteresting_root_keys, pb=None):
1330
    """Given root keys, find interesting nodes.
1331
1332
    Evaluate nodes referenced by interesting_root_keys. Ones that are also
1333
    referenced from uninteresting_root_keys are not considered interesting.
1334
1335
    :param interesting_root_keys: keys which should be part of the
1336
        "interesting" nodes (which will be yielded)
1337
    :param uninteresting_root_keys: keys which should be filtered out of the
1338
        result set.
1339
    :return: Yield
1340
        (interesting records, interesting chk's, interesting key:values)
1341
    """
1342
    # TODO: consider that it may be more memory efficient to use the 20-byte
1343
    #       sha1 string, rather than tuples of hexidecimal sha1 strings.
3735.2.68 by John Arbash Meinel
Add a TODO about avoiding all of the get_record_stream calls.
1344
    # TODO: Try to factor out a lot of the get_record_stream() calls into a
1345
    #       helper function similar to _read_bytes. This function should be
1346
    #       able to use nodes from the _page_cache as well as actually
1347
    #       requesting bytes from the store.
3735.9.19 by John Arbash Meinel
Refactor iter_interesting a little bit.
1348
1349
    # A way to adapt from the compressed texts back into fulltexts
1350
    # In a way, this seems like a layering inversion to have CHKMap know the
1351
    # details of versionedfile
1352
    adapter_class = versionedfile.adapter_registry.get(
1353
        ('knit-ft-gz', 'fulltext'))
1354
    adapter = adapter_class(store)
1355
1356
    (all_uninteresting_chks, all_uninteresting_items, interesting_keys,
1357
     interesting_records, interesting_items) = _find_all_uninteresting(store,
1358
        interesting_root_keys, uninteresting_root_keys, adapter, pb)
3735.9.7 by John Arbash Meinel
Cleanup pass.
1359
1360
    # Now that we know everything uninteresting, we can yield information from
1361
    # our first request
1362
    interesting_items.difference_update(all_uninteresting_items)
1363
    records = dict((record.key, record) for record in interesting_records
1364
                    if record.key not in all_uninteresting_chks)
3735.9.19 by John Arbash Meinel
Refactor iter_interesting a little bit.
1365
    if records or interesting_items:
1366
        yield records, interesting_items
3735.9.7 by John Arbash Meinel
Cleanup pass.
1367
    interesting_keys.difference_update(all_uninteresting_chks)
1368
1369
    chks_to_read = interesting_keys
3735.18.1 by John Arbash Meinel
Change the fetch logic to properly use the child_pb for child ops.
1370
    counter = 0
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1371
    while chks_to_read:
1372
        next_chks = set()
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1373
        for record in store.get_record_stream(chks_to_read, 'unordered', False):
3735.18.1 by John Arbash Meinel
Change the fetch logic to properly use the child_pb for child ops.
1374
            counter += 1
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1375
            if pb is not None:
3735.18.1 by John Arbash Meinel
Change the fetch logic to properly use the child_pb for child ops.
1376
                pb.update('find chk pages', counter)
3735.9.7 by John Arbash Meinel
Cleanup pass.
1377
            # TODO: Handle 'absent'?
3735.2.67 by John Arbash Meinel
Merge bzr.dev 3903 which brings in 'chunked' encoding.
1378
            if record.storage_kind in ('fulltext', 'chunked'):
1379
                bytes = record.get_bytes_as('fulltext')
1380
            else:
3735.9.17 by John Arbash Meinel
Pass around a progress bar and switch to using an adapter.
1381
                bytes = adapter.get_bytes(record,
1382
                            record.get_bytes_as(record.storage_kind))
3735.16.2 by John Arbash Meinel
Start passing around the search_key_func in more places.
1383
            # We don't care about search_key_func for this code, because we
1384
            # only care about external references.
1385
            node = _deserialise(bytes, record.key, search_key_func=None)
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1386
            if isinstance(node, InternalNode):
3735.9.15 by John Arbash Meinel
Found a bug in iter_interesting_nodes and its test suite.
1387
                chks = set(node.refs())
1388
                chks.difference_update(all_uninteresting_chks)
1389
                # Is set() and .difference_update better than:
1390
                # chks = [chk for chk in node.refs()
1391
                #              if chk not in all_uninteresting_chks]
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1392
                next_chks.update(chks)
1393
                # These are now uninteresting everywhere else
3735.9.7 by John Arbash Meinel
Cleanup pass.
1394
                all_uninteresting_chks.update(chks)
3735.9.19 by John Arbash Meinel
Refactor iter_interesting a little bit.
1395
                interesting_items = []
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1396
            else:
3735.9.19 by John Arbash Meinel
Refactor iter_interesting a little bit.
1397
                interesting_items = [item for item in node._items.iteritems()
1398
                                     if item not in all_uninteresting_items]
3735.9.15 by John Arbash Meinel
Found a bug in iter_interesting_nodes and its test suite.
1399
                # TODO: Do we need to filter out items that we have already
1400
                #       seen on other pages? We don't really want to buffer the
1401
                #       whole thing, but it does mean that callers need to
1402
                #       understand they may get duplicate values.
3735.9.7 by John Arbash Meinel
Cleanup pass.
1403
                # all_uninteresting_items.update(interesting_items)
3735.9.19 by John Arbash Meinel
Refactor iter_interesting a little bit.
1404
            yield {record.key: record}, interesting_items
3735.9.1 by John Arbash Meinel
Start working on an iter_interesting_nodes, which can find nodes to transmit
1405
        chks_to_read = next_chks