#59 Exec task output not complete

Current CVS
open-later
nobody
Core (11)
5
2011-10-20
2011-02-04
Anonymous
No

In the current implementation, when an exec task finishes the output and error logging threads are joined with a timeout of 2000ms.
This prove insufficient in situations where there is a lot of data that still need to be written (in my case an svn log --xml with a large number of revisions) and even if the task did finish.

The attached patch, while not perfect, resolve the situation simply by using the timeout parameter of the task to wait for logging threads to finish.
The good solution would be to review the way the logging is done as this test case highlight its inefficiency.

Regards,
Philippe

Discussion

  • Ryan Boggs
    Ryan Boggs
    2011-10-20

    The patch was backwards. Aside from that, I think setting the timeout to the default TimeOut out is too high a value. We'll look at this later.

     
  • Ryan Boggs
    Ryan Boggs
    2011-10-20

    • status: open --> open-later