yahoo / Oak

A Scalable Concurrent Key-Value Map for Big Data Analytics
Apache License 2.0
268 stars 48 forks source link

supporting JDK17 #179

Closed li0nr closed 2 years ago

li0nr commented 2 years ago

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Hey, I am opening this PR to allow Oak to work with JDK>=8, with minimal effort and without having to change a lot of code. While still using Unsafe and other sealed packages, though it is not encouraged anymore, and there exists some new alternatives that could replace them.

my changes introduce a new profile to the poms that we in order compile with JDK>= 8 and to run with JDK>=11 two tests fail when ran with JDK17, due to the changes introduced to the JDK past JDK8, which I need your help with (I still think there would be minimal effort to fix them)