[RFC] Clean up progress bars during fetch
Wouter van Heyst
larstiq at larstiq.dyndns.org
Wed Nov 8 17:05:26 GMT 2006
On Sat, Nov 04, 2006 at 11:57:06AM -0600, John Arbash Meinel wrote:
...
> TODO:
> The biggest thing that needs to be done at this point is to change the
> HTTP.readv() implementation. Right now it doesn a big download, then
> breaks up the file and yields the data. What we really need is for it
> to yield the data as it is read. sftp and local transports do this,
> and it means that we can get a meaningful progress while we are
> downloading inventory.knit to find what file ids are involved. If you
> want to see the difference use a machine that you have access over
> sftp and http, and you can see how the phase 1/6 over sftp has a nice
> updating progress, and over http it just hangs while it downloads all
> the data.
While I'd agree with yielding it earlier on, I can't see a difference
between sftp and http progress output at the moment?
> It would be really nice to have people play with this, and let me know
> if they think it is better than what we used to do. If people like it,
> then I'll spend the time to fix up any broken tests, and get it merged.
> (*I* like it, so I will probably do it anyway, but feedback always helps)
I think it's an improvement, though it could still be more finegrained.
> # Bazaar revision bundle v0.8
> #
> # message:
> # Fix bug in Knit.iter_lines_added_or_present_in_versions which was re-using a variable
> # committer: John Arbash Meinel <john at arbash-meinel.com>
> # date: Sat 2006-11-04 11:43:27.471999884 -0600
I like the result, but not really knowing how progress bars work, I must
confess not being sure what the changes were. Everything seems to move
up a level?
Wouter van Heyst
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 307 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20061108/518680fd/attachment.pgp
More information about the bazaar
mailing list