[MERGE][#294479] Fix many round-trips when pushing to pre-1.13 HPSS pack repository
John Arbash Meinel
john at arbash-meinel.com
Tue Mar 24 22:16:13 GMT 2009
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Andrew Bennetts wrote:
> This fixes two issues:
>
> * in bzr.dev, there's a regression vs. 1.13 causing lots of little append round
> trips when pushing to a pack repo on pre-1.13 smart server.
>
> * as a side-effect, this fixes the specific case that has been bothering people
> for a long time: that pushing to a *stacked* pack repository via pre-1.13
> HPSS is slow. That was <https://bugs.launchpad.net/bzr/+bug/294479>.
>
> In both cases upgrading the server would fix the issue (and give much better
> network performance in general), but within reason we don't want to make life
> too painful for people just because their server is a bit old.
>
> This patch is a little bit of a hack, but IMO it's not too outrageous. Thanks
> to Robert for some pointers about where best to place this hack.
>
> -Andrew.
>
>
There aren't any new tests that this is introducing, which seems a bit
odd. At a minimum, shouldn't we have a VF test that asserts that
'flush()' exists?
I suppose it is an Access issue, which does not already a per-XXX
implementation test.
So it seems that something like this should *somehow* generate a new
test. But at a minimum this seems better than what we have, so:
bb:approve
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iEYEARECAAYFAknJW60ACgkQJdeBCYSNAANwAwCgu3r6DP6DLMq+ka7Ves/YBchr
rKsAnRtbXO462Pq8wMRfdq3epF0wNk/a
=W4u/
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list