2 patches (http & paramiko)

[John A Meinel]

Robey Pointer wrote:
> I re-published by bzr.robey branch with 2 patches:
> 
> 1464: change http url parsing to use urlparse, and use the ui_factory to
> ask for a password if necessary
> 
> 1465: allow forcing the use of paramiko via environ var; use prefetch on
> paramiko >= 1.5.2
> 
> Prefetching turned out to have negligible effect during tests where I
> pushed a paramiko branch across my LAN.  It should help across
> high-latency links, but I haven't had a chance to test across such a
> link yet.
> 
> Compression across a LAN, on the other hand, cut my push time in half. 
> Posix users are already getting this for free because their pushes are
> going thru openssh.  Windows users may want to experiment with adding
> "t.use_compression()" here:
> 
> === modified file 'bzrlib/transport/sftp.py'
> --- bzrlib/transport/sftp.py
> +++ bzrlib/transport/sftp.py
> @@ -671,6 +671,7 @@
>          try:
>              t = paramiko.Transport((self._host, self._port))
> +            t.use_compression()
>              t.start_client()
>          except paramiko.SSHException:
>              raise SFTPTransportError('Unable to reach SSH host %s:%d' %
> (self._host, self._port))
> 
> 
> It's not well tested yet, so I didn't put that patch in my branch, but
> if it improves push times that much, it may be worth playing with.  I'm
> interested in hearing success/failure with it (even if via private email).
> 
> robey
> 

Interesting to hear about compression. Is it supported on all versions
of paramiko?

I know at some point in the future we might be going to a format which
is pre-compressed. So the gain might decrease. But certainly for now it
could be useful. Thanks for pointing it out.

John
=:->

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20051204/188bc657/attachment.pgp