[MERGE] Pyrex RIO implementation
Jelmer Vernooij
jelmer at samba.org
Thu May 14 21:10:06 BST 2009
On Thu, May 14, 2009 at 10:04:54PM +0200, Jelmer Vernooij wrote:
> On Thu, May 14, 2009 at 12:35:56PM -0500, John Arbash Meinel wrote:
> > You unfortunately don't get to share this buffer between Stanzas. You
> > *could*, though I'd want to make sure to memory cap it to something
> > reasonable, so that decoding a giant Revision text won't leave us with
> > lots of wasted memory just sitting around.
> I think the current improvements make things fast enough. Doing a
> malloc (there'll usually just be one or two per revision) doesn't seem
> like a huge problem to me.
Actually, I wrote that before I actually did it. It's in the patch.
With this patch, parsing all revision texts from bzr.dev now takes
~1.35s with the XML serializer and ~0.91s with the RIO one.
Cheers,
Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 315 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090514/35726dd9/attachment.pgp
More information about the bazaar
mailing list