You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by bu...@apache.org on 2011/10/24 15:23:10 UTC

DO NOT REPLY [Bug 52076] New: Add delay to retry task

https://issues.apache.org/bugzilla/show_bug.cgi?id=52076

             Bug #: 52076
           Summary: Add delay to retry task
           Product: Ant
           Version: nightly
          Platform: PC
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Core tasks
        AssignedTo: notifications@ant.apache.org
        ReportedBy: apache@avee.org
    Classification: Unclassified


Created attachment 27839
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=27839
Intial patch

I wanted the retry task to wait a bit before retrying. Since I didn't find a
clean way to do this I patched the retry task to support a 'retryDelay'
attribute. This simply sets the time to sleep between different retry attempts.
We currently use this on our build server to retry SVN updates which might fail
because multiple jobs are updating the same sandbox concurrently. So far it's
been working fine.

The attached patch has only been compiled/tested against java 1.6, but I guess
it should be fine in older versions as well.

The output, when a retryDelay is set looks like this:
updateLib:
     [echo] Start library update...
      [svn] svn: Working copy 'C:\BuildSystem\Libraries' locked; try performing
'cleanup'
      [svn] svn: Working copy 'C:\BuildSystem\Libraries' locked; try performing
'cleanup'
      [svn] <Update> failed.
    [retry] Attempt [0]: error occurred; retrying after 1000 ms...
      [svn] svn: Working copy 'C:\BuildSystem\Libraries' locked; try performing
'cleanup'
      [svn] svn: Working copy 'C:\BuildSystem\Libraries' locked; try performing
'cleanup'
      [svn] <Update> failed.
    [retry] Attempt [1]: error occurred; retrying after 1000 ms...
     [echo] Update done...

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 52076] Add delay to retry task

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=52076

Arjan Veenstra <ap...@avee.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         OS/Version|                            |All

--- Comment #1 from Arjan Veenstra <ap...@avee.org> 2011-10-24 13:23:47 UTC ---
I might update the patch which an increasing delay if there's a use for that.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 52076] Add delay to retry task

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=52076

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Stefan Bodewig <bo...@apache.org> 2012-02-01 12:47:46 UTC ---
added documentation with svn revision 1239108

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 52076] Add delay to retry task

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=52076

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|nightly                     |1.8.2
   Target Milestone|---                         |1.8.3

--- Comment #2 from Stefan Bodewig <bo...@apache.org> 2011-12-06 16:34:22 UTC ---
A slightly modified version of the patch is in as svn revision 1211004

It would be nice if you could add a patch for the manual and maybe even a test
as well.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.