Skip to content

Conversation

@yaooqinn
Copy link
Member

@yaooqinn yaooqinn commented Nov 5, 2025

What changes were proposed in this pull request?

Upgrade roaringbitmap from 1.3.0 to 1.5.3

Artifact updates at https://mvnrepository.com/artifact/org.roaringbitmap/RoaringBitmap have been stopped since 1.3.0. We need to add an additional repository, jitpack.io.

Why are the changes needed?

Dependency updates

Does this PR introduce any user-facing change?

No

How was this patch tested?

Passing GA

Was this patch authored or co-authored using generative AI tooling?

no

@yaooqinn
Copy link
Member Author

yaooqinn commented Nov 6, 2025

cc @dongjoon-hyun

@dongjoon-hyun
Copy link
Member

<enabled>false</enabled>
</snapshots>
</repository>
<repository>
Copy link
Contributor

@LuciferYang LuciferYang Nov 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Our previous concern was that the download source was flakiness: #49710 (comment)

If we can confirm that there's no issue with it, that would be great.

@yaooqinn
Copy link
Member Author

yaooqinn commented Nov 6, 2025

I didn't notice @LuciferYang 's PR. Thank you for providing me with the link, @dongjoon-hyun .

Let me merge this first to test the network traffic to the Jitpack servers. Considering the disparity in usage base between Jitpack and Maven Central, it is hoped this will not result in network-related exceptions.

Merged to master

@yaooqinn yaooqinn closed this in 4173bee Nov 6, 2025
@LuciferYang
Copy link
Contributor

I didn't notice @LuciferYang 's PR. Thank you for providing me with the link, @dongjoon-hyun .

Let me merge this first to test the network traffic to the Jitpack servers. Considering the disparity in usage base between Jitpack and Maven Central, it is hoped this will not result in network-related exceptions.

Merged to master

ok, let me close my pr

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants