[I] [Bug] ConcurrentBitmapSortedLongPairSet doesn't support pairs where the right value of the pair is larger than Integer.MAX_VALUE [pulsar]

["lhotari (via GitHub)" <gi...@apache.org>]

lhotari opened a new issue, #23877:
URL: https://github.com/apache/pulsar/issues/23877

   ### Search before asking
   
   - [x] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
   
   
   ### Read release policy
   
   - [x] I understand that unsupported versions don't get bug fixes. I will attempt to reproduce the issue on a supported version of Pulsar client and Pulsar broker.
   
   
   ### Version
   
   any released version
   
   ### Minimal reproduce step
   
   ConcurrentBitmapSortedLongPairSet uses internally RoaringBitmap which supports Integer values.
   
   
   ### What did you expect to see?
   
   Since the class interface and name is about supporting a Long pair, it should support long values for both right and left values of the pair.
   
   ### What did you see instead?
   
   Long value is not supported for the right value of the pair.
   
   ### Anything else?
   
   A possible solution would be to replace org.roaringbitmap.RoaringBitmap with org.roaringbitmap.longlong.Roaring64Bitmap. 
   However, Roaring64Bitmap interface is slightly different and would result in other changes. 
   
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug] ConcurrentBitmapSortedLongPairSet doesn't support pairs where the right value of the pair is larger than Integer.MAX_VALUE [pulsar]

["lhotari (via GitHub)" <gi...@apache.org>]

lhotari commented on issue #23877:
URL: https://github.com/apache/pulsar/issues/23877#issuecomment-2609980230

   #23878 has been merged and it supports values up to 2^32 (4294967296). This is 2 * (Integer.MAX_VALUE (2147483647) + 1) so there's an improvement that has been made..
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org