[BUNDLE] let PyCurl transport read multiple ranges.
michael at ellerman.id.au
Tue Jun 13 01:49:49 BST 2006
On Mon, 2006-06-12 at 11:44 +0200, Johan Rydberg wrote:
> "Michael Ellerman" <michael at ellerman.id.au> writes:
> >> Maybe we should go with my patch for now, and when your stuff is
> >> ready, we apply it? Better to get some functionality in now than wait
> >> forever for the "perfect patch" :)
> > I'd rather not, that original code I wrote is _really_ terrible. I'll
> > have something testable soon that's not so ugly.
> >> > Another thing we really need to do is some sort of detection of how
> >> > large a range header a server can handle. It looks like the default
> >> > size in Apache2 is ~8K per header (ie. the range string has to be <
> >> > than that), but it's configurable _downward_ by users. So it's quite
> >> > possible we'll hit servers with 4K or 2K or less.
> >> I tried sending 2000 ranges instead of 500, but actually lost
> >> performance for some reason. Did not investigate it further though.
> > Hmm, it probably ignored the range and sent you the whole file. Any
> > idea which server that was?
Cool. I've been getting 400 (bad request) because the Range header is
too big, but perhaps 2000 ranges is small enough to fit in the header
but large enough that Apache just gives up and sends the whole file.
phone: +61 2 6212 1183 (tie line 70 21183)
We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 191 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060613/5f324aaf/attachment.pgp
More information about the bazaar