RFR 8183123 : JDP packets have no processId context set

[Andrew Leonard]

Thanks Christoph,
Any offers for another reviewer of this fix please, it's only a few lines 
of code change.
Thanks
Andrew

Andrew Leonard
Java Runtimes Development
IBM Hursley
IBM United Kingdom Ltd
Phone internal: 245913, external: 01962 815913
internet email: andrew_m_leonard at uk.ibm.com 




From:   "Langer, Christoph" <christoph.langer at sap.com>
To:     "serviceability-dev at openjdk.java.net" 
<serviceability-dev at openjdk.java.net>
Date:   03/07/2017 08:25
Subject:        RE: RFR 8183123 : JDP packets have no processId context 
set
Sent by:        "serviceability-dev" 
<serviceability-dev-bounces at openjdk.java.net>



Hi,
 
Ping: Can we please get another review for this rather small JDK 10 fix?
 
Thanks
Christoph
 
 
From: serviceability-dev [
mailto:serviceability-dev-bounces at openjdk.java.net] On Behalf Of Andrew 
Leonard
Sent: Donnerstag, 29. Juni 2017 15:57
To: serviceability-dev at openjdk.java.net
Subject: RFR 8183123 : JDP packets have no processId context set
 
Hi All, 
Please can I get some review feedback for my changes for this issue: 
https://bugs.openjdk.java.net/browse/JDK-8183123 
The webrev patch has been uploaded here: 
http://cr.openjdk.java.net/~clanger/webrevs/8183123.0/ 

Essentially the fix entails: 
- Replacing invalid process id query logic with call to 
ProcessHandle.current().getPid(). 
- Update testcase to cover the failing scenario. Thus it fails without my 
patch, and succeeds with it. 

Thanks 
Andrew 

Andrew Leonard
Java Runtimes Development
IBM Hursley
IBM United Kingdom Ltd
Phone internal: 245913, external: 01962 815913
internet email: andrew_m_leonard at uk.ibm.com 

Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU


Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20170705/2ac37e2d/attachment.html>