-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
JDK8 doesn't supported? #514
Comments
Hi @catap, Java 8 is supported by 2.x versions which will still be actively maintained. Java 11 is required for 3.x in order to use VarHandles instead of requiring Unsafe. This is briefly mentioned in the release notes for v3. |
@ben-manes sorry, I've missed that. |
No problem! I added more emphasis in the release notes, as I can certainly understand the concern. |
but the refresh related improvements which were done in 3.0.0 will not be available in 2.x versions right? |
Correct. That was a large change to the feature which bordered on not being appropriate for a minor release. A major release would highlight that possibly incompatible changes were made which requires more diligence when upgrading. See for example #537 (comment) where a deadlock occurs because the user took advantage of the prior implementation not being linearizable, where returning the refresh future would block as part of a batching enhancement. This has to be changed to fit the new implementation. While it is unfortunate to see this bite someone, the major version bump gave a stronger warning of a potential break so it comes as less of a surprise. |
Hey,
I've tried to use the last release (3.0.0) with JDK8 and can't build it with error:
Is JDK8 supported? I haven't find any clear statement about it.
The text was updated successfully, but these errors were encountered: