-
Bug
-
Resolution: Fixed
-
Minor
-
None
-
-
2.372
after every build the processtree killer will kick off.
on my reasonably powererful windows laptop with lots of stuff open this can take 700+ms
This impacts every unit test and has been the cause of having to disable some tests on windows - which is not great.
The code needs to be improved.
One way to do this is to pass the list of environment variables to check down to C++ and only return the pids for them rather then every pid - which is then parsed and put into a map (which consumes over half of the time)
I have ~490 processes running,
- links to
[JENKINS-67681] hudson.util.ProcessTree$windows.killAll() is slow on windows
Description |
Original:
after every build the processtree killer will kick off.
on my reasonably powererful windows laptop with lots of stuff open this can take 700+ms This impacts every unit test and has been the cause of having to disable some tests on windows - which is not great. The code needs to be improved. One way to do this is to pass the list of environment variables to check down to C++ and only return the pids for them rather then every pid - which is then parsed and put into a map (which consumes over half of the time) !image-2022-01-26-15-29-32-660.png|thumbnail! |
New:
after every build the processtree killer will kick off.
on my reasonably powererful windows laptop with lots of stuff open this can take 700+ms This impacts every unit test and has been the cause of having to disable some tests on windows - which is not great. The code needs to be improved. One way to do this is to pass the list of environment variables to check down to C++ and only return the pids for them rather then every pid - which is then parsed and put into a map (which consumes over half of the time) !image-2022-01-26-15-29-32-660.png|thumbnail! I have ~490 processes running, |
Summary | Original: hudson.utul.ProcessTree$windows.killAll() is slow on windows | New: hudson.util.ProcessTree$windows.killAll() is slow on windows |
Assignee | New: James Nord [ teilo ] |
Status | Original: Open [ 1 ] | New: In Progress [ 3 ] |
Status | Original: In Progress [ 3 ] | New: Open [ 1 ] |
Status | Original: Open [ 1 ] | New: In Progress [ 3 ] |
Remote Link | New: This issue links to "upstream PR (Web Link)" [ 27376 ] |
Status | Original: In Progress [ 3 ] | New: In Review [ 10005 ] |
Remote Link | New: This issue links to "Jenkins PR to update winp (Web Link)" [ 27377 ] |
Released As | New: 2.372 | |
Resolution | New: Fixed [ 1 ] | |
Status | Original: In Review [ 10005 ] | New: Fixed but Unreleased [ 10203 ] |