[GitHub] [pulsar] codelipenghui commented on pull request #20176: [fix][broker] Fix `RoaringBitmap.contains` can't check value 65535

2023-05-17 Thread via GitHub
codelipenghui commented on PR #20176: URL: https://github.com/apache/pulsar/pull/20176#issuecomment-1550787809 @lhotari Yes, looks good to me. Thanks. -- 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

[GitHub] [pulsar] codelipenghui commented on pull request #20176: [fix][broker] Fix `RoaringBitmap.contains` can't check value 65535

2023-04-24 Thread via GitHub
codelipenghui commented on PR #20176: URL: https://github.com/apache/pulsar/pull/20176#issuecomment-1521184920 Interesting, ConcurrentBitmapSortedLongPairSet also used the same method. -- This is an automated message from the Apache Git Service. To respond to the message, please log