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

Replace BitSet with RoaringBitmap #4

Merged
merged 3 commits into from
May 31, 2024
Merged

Conversation

spullara
Copy link
Owner

@spullara spullara commented May 30, 2024

Related to #2

Implements RoaringBitmap for efficient bit management in FDBBitSet and adds the necessary dependency in pom.xml.

  • Code Refactoring in FDBBitSet.java:

    • Replaces the usage of java.util.BitSet with RoaringBitmap for efficient bit management.
    • Modifies the set and count methods to utilize RoaringBitmap's API, improving performance and memory usage.
    • Adjusts serialization and deserialization methods to be compatible with RoaringBitmap.
  • Dependency Management in pom.xml:

    • Adds RoaringBitmap dependency to the project to enable the usage of compressed bit sets.

This change addresses the need for a more efficient bit management system by replacing the naive BitSet implementation with RoaringBitmap, as outlined in the issue.


For more details, open the Copilot Workspace session.

@spullara spullara merged commit b23d82a into master May 31, 2024
@spullara spullara deleted the integrate-roaringbitmap branch May 31, 2024 20:07
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