Skip to content

bump roaringbitmap version #7734

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Jackie-Jiang merged 1 commit into from
Nov 10, 2021
Merged

bump roaringbitmap version #7734

Jackie-Jiang merged 1 commit into from
Nov 10, 2021

Conversation

@richardstartin
Copy link
Member

@richardstartin richardstartin commented Nov 9, 2021

Description

Updates RoaringBitmap to 0.9.23, which includes a performance improvement for range queries in some cases:

Benchmark                    (_dataType)  (_decile)  (_numDocs)                          (_scenario)  (_seed)  Mode  Cnt      Score     Error  Units
BenchmarkRangeIndex.queryV2         LONG          5    10000000                          NORMAL(0,1)       42  avgt    5   1535.029 ±  92.381  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                NORMAL(10000000,1000)       42  avgt    5   5785.355 ± 136.707  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                          EXP(0.0001)       42  avgt    5   8749.233 ± 611.313  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                             EXP(0.5)       42  avgt    5   3180.582 ±   9.839  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000              UNIFORM(0,100000000000)       42  avgt    5  20036.268 ± 521.648  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000  UNIFORM(100000000000, 100000000100)       42  avgt    5   3167.836 ± 270.252  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                   POWER(0,1000000,3)       42  avgt    5      2.687 ±   0.169  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                POWER(0,1000000000,1)       42  avgt    5      2.750 ±   0.344  us/op

Benchmark                    (_dataType)  (_decile)  (_numDocs)                          (_scenario)  (_seed)  Mode  Cnt      Score     Error  Units
BenchmarkRangeIndex.queryV2         LONG          5    10000000                          NORMAL(0,1)       42  avgt    5   1506.798 ±  35.190  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                NORMAL(10000000,1000)       42  avgt    5   5558.664 ± 125.293  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                          EXP(0.0001)       42  avgt    5   8785.747 ± 649.172  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                             EXP(0.5)       42  avgt    5   3226.282 ±  11.844  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000              UNIFORM(0,100000000000)       42  avgt    5  13849.725 ± 721.279  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000  UNIFORM(100000000000, 100000000100)       42  avgt    5   2097.835 ±  54.656  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                   POWER(0,1000000,3)       42  avgt    5      2.622 ±   0.053  us/op
BenchmarkRangeIndex.queryV2         LONG          5    10000000                POWER(0,1000000000,1)       42  avgt    5      2.668 ±   0.028  us/op

Upgrade Notes

Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

  • Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

  • Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

  • New configuration options
  • Deprecation of configurations
  • Signature changes to public methods/interfaces
  • New plugins added or old plugins removed
  • Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

@Jackie-Jiang Jackie-Jiang merged commit 928d4ef into apache:master Nov 10, 2021
kriti-sc pushed a commit to kriti-sc/incubator-pinot that referenced this pull request Dec 12, 2021
@richardstartin @kriti-sc
bump roaringbitmap version (apache#7734)
c1bb126
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Jackie-Jiang Jackie-Jiang Jackie-Jiang approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@richardstartin @Jackie-Jiang