twitter / finatra

Fast, testable, Scala services built on TwitterServer and Finagle

Home Page:https://twitter.github.io/finatra/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Upgrade Guice to 5.0.1+

greggdonovan opened this issue · comments

Is your feature request related to a problem? Please describe.

Finatra depends on Guice 4.2.x which depends on cglib which is no longer being developed and has not been updated to the new JDK9+ module system. See this Guice issue for more discussion.

Describe the solution you'd like

Upgrade Guice from 4.2.x to 5.0.1+.

Describe alternatives you've considered
With Guice 4.2.x's use of cglib, users need to add --add-opens java.base/java.lang=ALL-UNNAMED or something like it to their JVM startup options.

Additional context
N/A

@greggdonovan Thanks for the issue, please note that Finatra is still only targeted to work with JDK8 at the moment (and soon JDK11).

@cacoco Thanks! FWIW, we've been running Finatra for a year in production with JDK11 with no issues. This issue came up during testing of our JDK16 upgrade.

@greggdonovan that's great! Thanks again for the issue.

Any update on the timeline for next supported version of JDK?

@redkhalil we're targeting JDK11 support for Q1 or Q2 of 2022.