Remove class loader hack in `ReactiveStreamsOperatorsRecorder`

quarkusio / quarkus

Quarkus: Supersonic Subatomic Java.

https://quarkus.io

Apache License 2.0

13.64k stars 2.65k forks source link

Remove class loader hack in `ReactiveStreamsOperatorsRecorder` #43725

Open dmlloyd opened 1 day ago

dmlloyd commented 1 day ago

Description

The code has this comment:

    /**
     * ClassLoader hack to work around reactive streams API issue
     * see https://github.com/eclipse/microprofile-reactive-streams-operators/pull/130
     *
     * This must be deleted when Reactive Streams Operators 1.1 is merged
     */

The date is 2021-02-08. We appear to be using 3.0.1 of that project now, so presumably this would be safe to remove.

Implementation ideas

No response

dmlloyd commented 1 day ago

Relates to eclipse/microprofile-reactive-streams-operators#130.

dmlloyd commented 1 day ago

This applies to two classes:

io.quarkus.mutiny.reactive.operators.runtime.ReactiveStreamsOperatorsRecorder
io.quarkus.smallrye.reactivestreamsoperators.runtime.ReactiveStreamsOperatorsRecorder

dmlloyd commented 1 day ago

Found another one in io.quarkus.smallrye.openapi.runtime.OpenApiRecorder.