Skip to content

[SPARK-39626][BUILD] Upgrade RoaringBitmap from 0.9.28 to 0.9.30 - #37012

Closed
panbingkun wants to merge 2 commits into
apache:masterfrom
panbingkun:RoaringBitmap_upgrade
Closed

[SPARK-39626][BUILD] Upgrade RoaringBitmap from 0.9.28 to 0.9.30#37012
panbingkun wants to merge 2 commits into
apache:masterfrom
panbingkun:RoaringBitmap_upgrade

Conversation

@panbingkun

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This pr aims upgrade RoaringBitmap from 0.9.28 to 0.9.30 for fix bug.

Why are the changes needed?

This version contain bug fix: fix previousValue value smaller than first value

The changes between 0.9.28 and 0.9.30 as follows:

Does this PR introduce any user-facing change?

No

How was this patch tested?

Pass GA.

@LuciferYang

Copy link
Copy Markdown
Contributor

@panbingkun Could you run MapStatusesSerDeserBenchmark like #36719

@panbingkun

panbingkun commented Jun 28, 2022

Copy link
Copy Markdown
ContributorAuthor

@LuciferYang

Copy link
Copy Markdown
Contributor

Pr title should be [SPARK-39626][BUILD] Upgrade ...

@panbingkunpanbingkun changed the title [SPARK-39626][Build] Upgrade RoaringBitmap from 0.9.28 to 0.9.30 for fix bug[SPARK-39626][Build] Upgrade RoaringBitmap from 0.9.28 to 0.9.30Jun 28, 2022
@panbingkunpanbingkun changed the title [SPARK-39626][Build] Upgrade RoaringBitmap from 0.9.28 to 0.9.30[SPARK-39626][BUILD] Upgrade RoaringBitmap from 0.9.28 to 0.9.30Jun 28, 2022
@panbingkun

Copy link
Copy Markdown
ContributorAuthor

Pr title should be [SPARK-39626][BUILD] Upgrade ...

Updated

@LuciferYang

Copy link
Copy Markdown
Contributor

cc @dongjoon-hyun

@srowen

Copy link
Copy Markdown
Member

Merged to master

@AmplabJenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@panbingkun

Copy link
Copy Markdown
ContributorAuthor

Thank @LuciferYang & @srowen for review!

@panbingkun
panbingkun deleted the RoaringBitmap_upgrade branch July 1, 2022 01:52
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@panbingkun@LuciferYang@srowen@AmplabJenkins