1.6 fetch regression

[John Arbash Meinel]

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

Robert Collins wrote:
...

> Different code paths:
> pack->pack is Packer.pack()
> remote->pack is fetch()
> 
> I've been working at making remote->pack better but its obviously not
> there yet.

Actually, it is Packer.pack()

Specifically:
class InterRemoteToOther(InterRepository):
...
    def fetch(self, revision_id=None, pb=None, find_ghosts=False):
        self._ensure_real_inter()
        return self._real_inter.fetch(revision_id=revision_id, pb=pb,
            find_ghosts=find_ghosts)

^- self._real_inter is InterPackRepo, so it also triggers the Packer.pack() code.

And sftp is *slower* than bzr+ssh, because it has even more overhead for stuff.


> 
>> Over the local network I also see:
>> $ time bzr.dev branch http://
>>   35.41s user 3.40s system 69% cpu 55.926 total
>> $ time bzr1.5 branch http://
>>   38.61s user 3.30s system 69% cpu 1:00.54 total
>>
>> (note that this repo is slightly different, but still a packed repo) I'm
>> actually quite surprised to see that bzr-1.5 branching over http:// is
>> *faster* than branching locally. (1m versus 1m15s).
> 
> async behaviour in pycurl, I'd guess.
> 
> -Rob

I'm using urllib.

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

iD8DBQFItfJ8JdeBCYSNAAMRAvu/AKC2s7ep8HAMrI077OKk5vfR8eEaOACffxuZ
9UiAdmG8aNUcsk32s8jShbE=
=ObV/
-----END PGP SIGNATURE-----