nitrite / nitrite-java

NoSQL embedded document store for Java
https://bit.ly/no2db
Apache License 2.0
825 stars 95 forks source link

feat: graalvm support for nitrite-mvstore-adapter #995

Closed DarkAtra closed 2 months ago

DarkAtra commented 2 months ago

Fixes: https://github.com/nitrite/nitrite-java/issues/403

Note: this PR does not make nitrite fully native compatible as i did not test all codepaths. To be tested:

Already tested via nitrite-native-tests:

anidotnet commented 2 months ago

@DarkAtra thanks a lot for the PR. Are you planning to complete the To be tested: items also? Please include rocksdb-adapter in your To be tested: list.

DarkAtra commented 2 months ago

@DarkAtra thanks a lot for the PR. Are you planning to complete the To be tested: items also? Please include rocksdb-adapter in your To be tested: list.

I don't think that i'm going to complete all items on the todo list as it's quite a lot of work. However, with this PR, it should be easier to add and test native hints for remaining modules in the future. It might also help to migrate all existing tests to JUnit 5 first and then continue with GraalVM compatibility as this would allow reusing them.