RFR: 8168414 Various timeouthandler fixes

Marcus Larsson marcus.larsson at oracle.com
Thu Oct 20 14:17:27 UTC 2016


On 10/20/2016 03:27 PM, Staffan Larsen wrote:
> When looking for some timeout handler problems I found a few things that should be fixed:
> * Strange use of Thread.currentThread().interrupt()
> * Add logging for timeouts
> * Milliseconds sometimes printed as microseconds or nanoseconds
> * Should use destroyForcibly() to terminate processes
> * No need to sleep in the last iteration when running a command multiple times
> * Disable timeout handling timeouts since we do that ourselves
> I didn’t want to file individual bugs for all of these, so I have lumped them together in one bug.
> bug: https://bugs.openjdk.java.net/browse/JDK-8168414 <https://bugs.openjdk.java.net/browse/JDK-8168414>
> webrev: http://cr.openjdk.java.net/~sla/8168414/webrev.00 <http://cr.openjdk.java.net/~sla/8168414/webrev.00>

Looks good to me!


> Thanks,
> /Staffan

More information about the build-dev mailing list