Alan, I have updated the review by removing certain tests from the 'src' directory (which I believe are obsolete). The SQE will be removing the tests owned by them separately, they have already quarantined the relevant tests.
http://cr.openjdk.java.net/~sballal/8158050/webrev.02/index.html -Sharath Ballal From: Alan Bateman Sent: Sunday, July 10, 2016 2:04 PM To: Sharath Ballal; serviceability-dev@openjdk.java.net Subject: Re: RFR: 8158050: Remove SA-JDI On 08/07/2016 05:57, Sharath Ballal wrote: Hello, Please review the fix for: HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8158050"JDK-8158050 - Remove SA-JDI The webrev is at: HYPERLINK "http://cr.openjdk.java.net/%7Esballal/8158050/webrev.01/index.html"http://cr.openjdk.java.net/~sballal/8158050/webrev.01/index.html The fix is for removing the SA-JDI code and jsadebugd. Jsadebugd is being moved to jhsdb as part of another bug (review will be sent next). This looks good. One surprise is that I assumed there would be more updates or removal of tests needed but it seems not to be the case. -Alan
