ant-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 50229] Slow down in StreamPumper
Date Thu, 16 Dec 2010 08:49:47 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=50229

--- Comment #5 from Stefan Bodewig <bodewig@apache.org> 2010-12-16 03:49:44 EST ---
The change is necessary in order to properly deal with grandchild processes
on Windows - that's why it used to be explicitly restricted to Windows
initially.

If you take out the change the AntUnit test testDoesntWaitForChildren in
exec-test is going to take more than 20 seconds because Ant now waits
for the grandchildren the forked (and long finished) child process
has spawned.

So reverting the change is no option.  Finding a different solution for
bug 5003 would be one, but so far nobody has found one.

One option may be to not set useAvailable to true if we know the forked
process is not going to spawn additional child prcesses - like in <attrib>'s
case and make it configurable via attributes on <exec> and friends.

One thing to consider is that a while back the useAvailable flag has
been set to true on all operating systems because it seemed to fix
another bug.  I don't recall which bug it has been and don't have the time
to search for it right away, but I recall it was not related to grandchildren
at all - so this bug might reappear with my idea above.  Then again I also
recall somebody (I think Jesse) questioned whether the bug has really been
fixed and whether the change was the correct one at all.

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

Mime
View raw message