Hi Peter,

On 24/10/2014 11:39 PM, Peter Allwin wrote:
Hello!

This patch fixes two intermittent issues seen over the past year:

   a) Possible failure where an existing pid-file is not owned by the
test user
   b) Race during startup where we try to attach to the target before
it’s ready (removed arbitrary 5sec sleep)

Bug: https://bugs.openjdk.java.net/browse/JDK-8024055
Webrev: http://cr.openjdk.java.net/~allwin/8024055/webrev.00/

test/serviceability/attach/AttachWithStalePidFile.java

Why use an Executor and a Future instead of simply processing the process output directly?

David

Tested locally on my Mac.

Thanks!
/peter

Reply via email to