> Hi all, > > Test java/lang/ProcessBuilder/PipelineLeaksFD.java intermittent timed out, > because `lsof` invoke huast lots of time when the tested machine has many > processes, and the processes open too many files. > > This PR add parameter -p pid to `lsof`, which will only generate output from > the wanted processes, rather than all the processes on the machine, this will > make `lsof` use less time to finish significantly. And this PR also use > `Process.waitFor(long timeout, TimeUnit unit)` instead of `waitFor()` which > will avoid waitFor invoke cause test timed out. Delete the history lsof input > and output files will make diagnosis more easy. > > Change has been verifed locally. The imtermittent timed out do not observed > anymore.
SendaoYan has updated the pull request incrementally with one additional commit since the last revision: Remove retry on destroyForcibly ------------- Changes: - all: https://git.openjdk.org/jdk/pull/29478/files - new: https://git.openjdk.org/jdk/pull/29478/files/c4522418..ad1dd3b3 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=29478&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29478&range=01-02 Stats: 4 lines in 1 file changed: 0 ins; 4 del; 0 mod Patch: https://git.openjdk.org/jdk/pull/29478.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/29478/head:pull/29478 PR: https://git.openjdk.org/jdk/pull/29478
