Skip to content
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

Update primary dependencies in db-shootout benchmark to improve JVM compatibility #412

Merged
merged 5 commits into from
Oct 2, 2023

Conversation

lbulej
Copy link
Member

@lbulej lbulej commented Sep 30, 2023

Updates MapDB version from 3.0.1 to 3.0.10, H2 MVStore from version 1.4.192 to version 2.1.214, and Chronicle Map from version 3.17.0 to version 3.22.9

This makes the db-shootout benchmark compatible with JDK 18 (up from 11).

Starting with JDK 19, the benchmark fails because chronicle-core is using reflection to look up the sun.nio.ch.FileChannelImpl.map0(int,long,long,boolean) method which is not available in JDK 19 anymore.

Using more recent Chronicle Map (such as 3.23.5 or even 3.24ea3) is currently not an option, because SBT fails to find their third-party-bom.

Also drops the jcl-over-slf4j version override and instead adds an
override for slf4j-api version.
Updates MapDB version from 3.0.1 to 3.0.10
Updates H2 MVStore from version 1.4.192 to version 2.1.214
Updates Chronicle Map from version 3.17.0 to version 3.22.9

The db-shootout now supports JVM version from 8 to 18 instead of
8 to 11.

JVM 19 is not supported because chronicle-core uses reflection to look
up the sun.nio.ch.FileChannelImpl.map0(int,long,long,boolean) method
which is no longer present in JDK19. SBT fails to find third-party-bom
for more recent Chronicle Map versions.
@lbulej lbulej requested a review from vhotspur October 1, 2023 16:18
@lbulej lbulej marked this pull request as ready for review October 1, 2023 16:18
Copy link
Member

@vhotspur vhotspur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on OpenJDK 64-Bit Server VM (build 18.0.1.1+2-6, mixed mode, sharing), looks good. Thanks!

@lbulej lbulej merged commit ae3b5ce into master Oct 2, 2023
15 checks passed
@lbulej lbulej deleted the devel/database branch October 2, 2023 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants