<threads:waitFor> does not use onlyWait as maximum wait time for thread groups
------------------------------------------------------------------------------

         Key: JELLY-174
         URL: http://nagoya.apache.org/jira/browse/JELLY-174
     Project: jelly
        Type: Bug
    Reporter: Matthias Kerkhoff


Consider 
  <threads:waitFor group="${tg}" onlyWait="60000" status="SUCCESS"/>

How long will this tag wait at most? Well, it depends. It depends on how many 
threads are in the group. The worst case will be onlyWait * tg.size(). Each 
thread, however, may not take longer than onlyWait to complete.

>From my experience when a timeout like onlyWait is needed, one usually wants 
>to set an upper limit for the _group_ as a whole. How this timeout is spread 
>among the threads in the group often doesn't matter. 
In the current implementation, the limit is a per-thread timeout even if a 
group is specified.

I would suggest a small code change, which changes the behaviour of waitFor in 
a way that when a group is specified the timeout specifies the time that _all_ 
threads in the group may take to end or to reach the specified status.

The old behaviour can easily be achieved by iterating over each thread in the 
group and calling <threads:waitFor> for that thread.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://nagoya.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to