jetty.project icon indicating copy to clipboard operation
jetty.project copied to clipboard

Jetty 11 and Micrometer

Open tipsy opened this issue 3 years ago • 2 comments

Background

While migrating Javalin to Jetty 11, we discovered that Micrometer doesn't support Jetty 11. We made a ticket with the Micrometer team: https://github.com/micrometer-metrics/micrometer/issues/3234, but adding support isn't straightforward.

In short, Micrometer is not able to support Jetty 11 without dropping support for Jetty 9 (not feasible), or creating a separate module (not desirable).

Feature request

It would be nice if there existed a jetty-micrometer or micrometer-jetty artifact. The Micrometer team prefers that the Jetty team provides this artifact, and I guess the Jetty team would prefer that the Micrometer team provides this. I don't know which option makes the most sense, but the Javalin team would be very happy if Micrometer and Jetty 11 could work together, so here I am.

tipsy avatar Jul 29 '22 12:07 tipsy

We are not opposed to having 3rd party integrations in jetty where in makes sense. But hard to say we will do it without much knowledge of micrometer. Perhaps the micrometer team could prepare a PR against jetty that would show what a "handed over" integration would look like.

gregw avatar Jul 29 '22 13:07 gregw

Many projects are moving forward and dropping support for Java 8 and requiring at minimum Java 11. Micrometer will have the same problem you report here when they will have to integrate Tomcat 10.1, which requires Java 11, or other projects Micrometer depends on (kafka, etc.). It's just that Jetty 11 has bitten that bullet before.

Eventually Micrometer could decide to keep the 1.x releases for Java 8, Jetty 9, Tomcat 9, etc. and have a 2.x release for Java 11, Jetty 11, Tomcat 10.1, and so on.

We know the pain of maintaining multiple branches for multiple dependencies and Java versions and we sympathize with Micrometer, but we cannot maintain integrations for other projects for every project out there.

sbordet avatar Jul 29 '22 13:07 sbordet

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jul 30 '23 00:07 github-actions[bot]

This was resolved by the Micrometer team.

tipsy avatar Jul 30 '23 00:07 tipsy