2 patches (http & paramiko)

Robey Pointer robey at lag.net
Sun Dec 4 22:51:32 GMT 2005


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





More information about the bazaar mailing list