[Duplicity-tracker] [patch #6652] improve asynch scheduler (including the synchronous case)

[Peter Schuller]

URL:
  <http://savannah.nongnu.org/patch/?6652>

                 Summary: improve asynch scheduler (including the synchronous
case)
                 Project: duplicity
            Submitted by: scode
            Submitted on: Tue 14 Oct 2008 10:29:37 PM GMT
                Category: None
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any

    _______________________________________________________

Details:

I realized there are a couple of left-overs in the current AsyncScheduler,
that also affects the non-asynchronous case.

If the execution of task (in practice, an upload) fails, that failure is not
communicated to the front-end until all uploads have been attempted. This is
not good because it ends up wasting time and bandwidth.

The attached patch fixes this case.

In addition, for the case of --asynchronous-upload, the patch adds support
for failing early in that case as well (though in one edge case it may get
delayed one task more than it might have to).

THIRDLY, a typo prevented the creation of more than 1 worker (which is not a
practical problem now since we do not yet support more than 1).

However, in fixing that I also had to add the proper concurrency limitation
which was not yet in place.

So in short, various improvements to the asynch scheduler, with the most
important fix being that which affects synchronous upload (when running
without --asynchronous-upload).




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Tue 14 Oct 2008 10:29:37 PM GMT  Name: duplicity_asynchfixes.patch 
Size: 7kB   By: scode

<http://savannah.nongnu.org/patch/download.php?file_id=16675>

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?6652>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/