[MERGE/RFC] Versioned properties spec

[Alexander Belchenko]

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

Aaron Bentley пишет:
> Alexander Belchenko wrote:
>> Alexander Belchenko ?8H5B:
>>> here is my proposal for versioned properties.
>> I apologize for my poor English skill.
>> Here is slightly reworked version.
> 
> bb:resubmit
> 
> +Serialization
> +-------------
> +VP stored inside inventory. For serialization is proposed to using bencode.
> 
> I would like to see a lot more detail than that.  Especially, I'd like
> to understand how you intend to store them in the repository.  Bencoded?
>  Why not using xml?  It's quite good at key/value pairs.

I think we need to avoid zoo of serialization formats. It's complicate
implementation and makes code less maintainable.

> +Format
> +~~~~~~
> + Each section name is path of file relative to
> +WT root.
> 
> This is somewhat brittle; if a file is moved before the HRF version is
> updated, the association with path will be lost.

It's OK because HRF is not intended to be in sync with internal data all the time.
When user generate dump of VP bzr will produce [read-only] dump. If user wants
to import HRF back and update internal data, he should take care about renames
(or just don't rename files until VP will be updated).
Using file ids instead of relative paths make HRF less human-friendly.

> +Keep defaults in properties of directory entries
> +------------------------------------------------
> 
> This seems like a strange idea to me.  I would expect most properties to
> be grouped by file type, not directory location.
> 
> Also, this is not our usual access pattern and may not be performant.

So I shrink my idea to keep defaults at tree root level only. It's possible
to extend it later.


Thank you again for English corrections.
New version attached.

- --
[µ]

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFG3NbozYr338mxwCURAgaHAKCSo94lrgomg/29VuxUKt3h0VpHHACfVem7
sxGn5htHoVv2SWrR+i4pwPk=
=4f5X
-----END PGP SIGNATURE-----
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: vp-spec.v2.patch
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20070904/3b006fb2/attachment.diff