Skip to content

[improve] Upgrade RoaringBitmap to 1.6.6 version#25253

Draft
lhotari wants to merge 1 commit intoapache:masterfrom
lhotari:lh-upgrade-roaringbitmap
Draft

[improve] Upgrade RoaringBitmap to 1.6.6 version#25253
lhotari wants to merge 1 commit intoapache:masterfrom
lhotari:lh-upgrade-roaringbitmap

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Feb 19, 2026

Motivation

There have been improvements and bug fixes in RoaringBitmap since 1.2.0 version. We haven't been able to upgrade since the artifact hasn't been published to maven central for a while. RoaringBitmap is now available in maven central with coordinates org.roaringbitmap:roaringbitmap this has changed from the previous org.roaringbitmap:RoaringBitmap dependency. Pulsar users using pulsar-client-original without shading might need to pay attention to this detail if they are depending on org.roaringbitmap:RoaringBitmap directly since that would cause 2 versions of the library on the classpath simultaneously. Either one would be used.

Modifications

  • upgrade RoaringBitmap to 1.6.6 version

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari added this to the 4.2.0 milestone Feb 19, 2026
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Feb 19, 2026
@lhotari lhotari marked this pull request as draft February 19, 2026 21:27
@lhotari
Copy link
Member Author

lhotari commented Feb 19, 2026

Let's wait for RoaringBitmap/RoaringBitmap#824

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.59%. Comparing base (436ca72) to head (19128c9).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #25253      +/-   ##
============================================
- Coverage     72.75%   72.59%   -0.16%     
+ Complexity    34068    33888     -180     
============================================
  Files          1959     1959              
  Lines        155401   155401              
  Branches      17724    17724              
============================================
- Hits         113062   112818     -244     
- Misses        33364    33590     +226     
- Partials       8975     8993      +18     
Flag Coverage Δ
inttests 25.64% <ø> (-0.32%) ⬇️
systests 22.35% <ø> (-0.10%) ⬇️
unittests 73.55% <ø> (-0.17%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 74 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments