Skip to content

[SPARK-55616][BUILD] Bump RoaringBitmap 1.6.10 which backs to Maven Central#54393

Open
pan3793 wants to merge 2 commits intoapache:masterfrom
pan3793:SPARK-55616
Open

[SPARK-55616][BUILD] Bump RoaringBitmap 1.6.10 which backs to Maven Central#54393
pan3793 wants to merge 2 commits intoapache:masterfrom
pan3793:SPARK-55616

Conversation

@pan3793
Copy link
Member

@pan3793 pan3793 commented Feb 20, 2026

What changes were proposed in this pull request?

Notable changes compared to 1.6.0,

Why are the changes needed?

Keep third-party dependency up to date, and remove third-party Maven repo - JitPack, which is slow sometimes.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GHA.

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

No.

@LuciferYang
Copy link
Contributor

LuciferYang commented Feb 20, 2026

https://central.sonatype.com/artifact/org.roaringbitmap/RoaringBitmap

image

This is a good change, but I've noticed that the latest version has just been updated to 1.6.10. We'd better wait and upgrade after the new version becomes stable.

@pan3793 pan3793 changed the title [SPARK-55616][BUILD] Bump RoaringBitmap 1.6.9 which backs to Maven Central [SPARK-55616][BUILD] Bump RoaringBitmap 1.6.10 which backs to Maven Central Feb 20, 2026
@pan3793
Copy link
Member Author

pan3793 commented Feb 20, 2026

@LuciferYang, I checked the commit history between 1.6.9 and 1.6.10, all changes are about building and publishing.

I updated to use 1.6.10, and all tests pass now, please let me know if you want to wait for more time before merging.

This is a good change, but I've noticed that the latest version has just been updated to 1.6.10. We'd better wait and upgrade after the new version becomes stable.

@LuciferYang
Copy link
Contributor

Let's wait for two days. If no new fix is released, I think we can merge it.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM for (1) moving back and (2) waiting for a few more days.

cc @yaooqinn , too.

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.

3 participants

Comments