Comment 3 for bug 374738

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 374738] Re: should stream an inventory delta for xml to chk inventory fetching

On Mon, 2009-06-29 at 01:28 +0000, Andrew Bennetts wrote:
>
> - look up the parent inventory in the target
> - apply the delta
> - add_inventory

This is one call: repository.add_inventory_by_delta

> - (otherwise, use get_bytes_as('fulltext') as before)
> - update StreamSource to generate deltas rather than fulltexts when
> going from chk->non-chk, or vice-versa. (Or maybe just cross format
> in general?)

Cross format in general IMO.

> - update the network protocol
> - serialise & deserialise inventory-delta records, e.g. update
> NetworkRecordStream. (bzrlib.inventory_delta already exists and will
> do most of the heavy lifting)
> - perhaps need new insert_stream and get_stream verbs to gracefully
> handle backwards compatibility.

Likely yes. We should try to do progress at the same time (by which I
mean before considering the verb stable in trunk). Or something.

-Rob