search

opentracing-contrib / java-jaxrs

OpenTracing Java JAX-RS instrumentation
Apache License 2.0
37 stars 33 forks source link

Update JAX-RS to 2.1 #52

Open hypnoce opened 6 years ago

hypnoce commented 6 years ago

Hi,

as requested in #49, JAX-RS was not updated to the latest version 2.1. Do you think it brings value to do so ?

Thanks !

Francois

tmszdmsk commented 6 years ago

@hypnoce IMHO if there are no features we want to use in JAX-RS integration and it is backwards compatible (it's sane to assume it is) then there is no benefit of doing so. It can accidentally bump the version of JAX-RS in projects using it when resolving version.

hypnoce commented 6 years ago

Indeed it is premature to do so now.

Nevertheless, the jaxrs dependency is declared as provided. Therefore there should not be any bump accident.

I think it makes more sense to work on this issue when upgrading the test dependencies to validate this instrumentation to newer versions of the jaxrs implementations.

jpkrohling commented 6 years ago

Anyone up to speed with the current state of JAX-RS ? Is there anything that we'd need to do, or can this one be closed?