[MERGE][bug #300177] Pass compression_parent as part of 'get_record_stream'

John Arbash Meinel john at arbash-meinel.com
Tue Dec 16 23:34:08 GMT 2008


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

The attached patch does 2 basic things:

1) Changes the api so that the records (ContentFactory objects) from
get_record_stream have a member called "compression_parents".

2) Change insert_record_stream to use the compression_parents field, and
take appropriate action.


I went for the plural form, because it doesn't require a code change if
we ever do multi-parent deltas.

At the moment, pack format repositories could actually handle a delta
which does not point at the left-hand parent. You can get this via "bzr
reconcile" after the per-file graph was a bit wonky (we had this in
older versions of bzr).

I originally was planning on changing insert_record_stream() to preserve
the fact that it was a non left-hand-parent. However, that required a
lot of updates, and means that the repository wouldn't be compatible
with older clients. So rather than spend a lot of time getting that
working, I just punted and always extract to a fulltext and re-insert.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAklIOvAACgkQJdeBCYSNAAMbvgCgtZZLFr3WvCjvymJCOGh4CcXj
x3EAnjc4+yz+LfU5ypzGegQwfhKb1IEr
=GHJ/
-----END PGP SIGNATURE-----
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 300177_record_stream_compression_parents.patch
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20081216/30789ec8/attachment-0001.diff 


More information about the bazaar mailing list