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

feat(#651): Java module system support, upgrade to Java 11 #656

Closed
wants to merge 2 commits into from

Conversation

novoj
Copy link

@novoj novoj commented Sep 20, 2023

Added a module-info.java file describing the public interface of the RoaringMap. Since all classes are in a single package, it represents practically all content.

Previous compatibility with Java 8 and the shims library, which contained a multi-release build supporting both Java 11 and older versions of the Java runtime, could not be maintained. Currently only Java 11 is supported by RoaringBitmap.

BREAKING CHANGE: Minimal Java version updated to Java 11.

SUMMARY

  • Describe your changes, including rationale and design decisions
  • If your pull request includes new functionality, then you must have corresponding new tests.
  • We require all Java files to be ASCII-only to avoid compatibility problems.

Automated Checks

  • I have run ./gradlew test and made sure that my PR does not break any unit test.
  • I have run ./gradlew checkstyleMain or the equivalent and corrected the formatting warnings reported.

Added a `module-info.java` file describing the public interface of the RoaringMap. Since all classes are in a single package, it represents practically all content.

Previous compatibility with Java 8 and the `shims` library, which contained a multi-release build supporting both Java 11 and older versions of the Java runtime, could not be maintained. Currently only Java 11 is supported by RoaringBitmap.

BREAKING CHANGE: Minimal Java version updated to Java 11.
Corrected duplicated imports. Added `art` package to opened packages since its used in RoaringBitmap64 API (method parameters).
@novoj novoj closed this Sep 21, 2023
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.

1 participant