ring

RAJKUMAR 2024-04-02T03:21:50.931119Z

why ring-jetty-adapter doesn't support jetty 12 yet?

Eugen 2024-04-21T21:19:15.211559Z

I've also just fit some issues with Java 21 and sunng87 adapter. Trying to fix the dependecies (but woudl like to avoid). What is the current path forward for Java 21 and ring ?

seancorfield 2024-04-21T23:01:12.363519Z

We use Ring with Jetty 11 on jdk 21. We switched back from the sunng87 adapter when Ring updated to Jetty 11.

seancorfield 2024-04-21T23:01:54.821329Z

Jetty 12 is problematic but JDK 21 is fine.

seancorfield 2024-04-21T23:03:58.693099Z

And, for us, virtual threads are off the table until the MySQL JDBC driver becomes compatible with them (it relies on synchronized far too much).

weavejester 2024-04-02T12:10:51.366589Z

Jetty 12 also requires Java 17, while Jetty 11 requires Java 11. Ring is deliberately conservative in its approach, aiming for maximum compatibility. This is why we stuck with Jetty 9.2 for so long; because it only required Java 1.7. When Jetty 9.2 was deprecated, we moved up to Jetty 11, skipping Jetty 10 as the requirements were the same.

👍 1
Eugen 2024-04-22T07:03:53.624159Z

thanks for sharing @seancorfield

Eugen 2024-04-22T09:21:51.192589Z

I filed an issue (see bellow) since I can't do the quickstart using ring-jetty-adapter and ring 1.12.1 https://github.com/ring-clojure/ring/issues/503

seancorfield 2024-04-02T03:26:47.395359Z

Jetty 12 has substantial changes from Jetty 11 -- Substantial enough that we had to back off trying to use it (via the sunng87 adapter) because observability via New Relic is terrible. The sunng87 adapter has to maintain two separate branches: one for Jetty 11 and one for Jetty 12.

RAJKUMAR 2024-04-02T03:27:45.626149Z

ohh okay

RAJKUMAR 2024-04-02T03:27:58.160519Z

Thanks @seancorfield!!!