RFR 8068976: Remove JSDT implementation

[Jaroslav Bachorik]

On 14.1.2015 15:13, Keith McGuigan wrote:
> What evidence do you have that no OpenJDK users are using this?

A search from the snapshot of the maven central as of May 2014 finds
only no reference to com.sun.tracing except it finds one artifact
jsdt-compat (https://github.com/cmbntr/jsdt-compat) that redistributes
com.sun.tracing APIs and implementation classes for pre-JDK7 usage.

As there is no possible way to use JSDT without going through 
com.sun.tracing API the lack of references in maven central could be 
good enough indication of this functionality not being in use.

AFAIK, there is no other reliable way to figure out whether an API is in 
active use or not :(

-JB-

>
> On Wed, Jan 14, 2015 at 9:05 AM, Jaroslav Bachorik
> <jaroslav.bachorik at oracle.com <mailto:jaroslav.bachorik at oracle.com>> wrote:
>
>     Please, review the following removal of functionality.
>
>     Issue : https://bugs.openjdk.java.net/__browse/JDK-8068976
>     <https://bugs.openjdk.java.net/browse/JDK-8068976>
>     Webrev: http://cr.openjdk.java.net/~__jbachorik/8068976/webrev.00
>     <http://cr.openjdk.java.net/~jbachorik/8068976/webrev.00>
>
>     JDK-8062303 [1] requests the removal of com.sun.tracing API. This
>     API was added as experimental in JDK7 [2][3] and haven't seen any
>     significant uptake since then.
>
>     JSDT builds on top of the com.sun.tracing API and provides the
>     bridge to DTrace. It allows Java programs to fire custom DTrace
>     probes. As an implementation of com.sun.tracing API it needs to be
>     removed before proceeding with the removal of that API.
>
>     This change is basically an anti-delta to [4] - minus the changes
>     not related to JSDT and/or DTrace.
>
>     This change passes all the regression and JCK tests.
>
>     Thanks,
>
>     -JB-
>
>
>     [1] https://bugs.openjdk.java.net/__browse/JDK-8062303
>     <https://bugs.openjdk.java.net/browse/JDK-8062303>
>     [2] https://bugs.openjdk.java.net/__browse/JDK-6537506
>     <https://bugs.openjdk.java.net/browse/JDK-6537506>
>     [3] https://bugs.openjdk.java.net/__browse/JDK-7037081
>     <https://bugs.openjdk.java.net/browse/JDK-7037081>
>     [4]
>     http://hg.openjdk.java.net/__jdk9/hs-rt/hotspot/rev/__018d5b58dd4f
>     <http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/018d5b58dd4f>
>
>
>
>
> --
>
> twitter-icon-large.png
>
> 	
>
> Keith McGuigan
>
> @kamggg
>
> kmcguigan at twitter.com <mailto:kmcguigan at twitter.com>
>