[Yum-devel] [PATCH] Group MD downloads, and use the length to give total size.

James Antill james at fedoraproject.org
Tue Oct 13 04:27:41 UTC 2009


On Mon, 2009-10-12 at 17:45 -0400, Seth Vidal wrote:

> as before NACK pre-3.2.25 release

 *nods*

> but I have to ask - how is this on the road to multiple md downloads at 
> once? The way curlmulti works we just toss objects  at it and perform the 
> ones we want and cope with the callback.
> 
> How does this help us do that?

 Because now there's a "download section" which just does:

        for (ndata, nmdtype) in downloading*:
            if not self._retrieveMD(nmdtype, retrieve_can_fail=True):
                self._revertOldRepoXML()
                return False

...instead of having it mixed in, and in theory it just needs to change
to something like:

        for (ndata, nmdtype) in downloading*:
            self._start_async_retrieveMD(nmdtype, retrieve_can_fail=True)
        if not self._end_async():
            self._revertOldRepoXML()
            return False

...of course that's pure guess work, and we might want to change it a
lot more so you can have multiple downloads for different repos. going
at once ... but, it's something.



More information about the Yum-devel mailing list