Skip to content
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.

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

Simplify Routes/RoutePattern trees #3409

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

987Nabil
Copy link
Contributor

@987Nabil 987Nabil commented Mar 16, 2025

fixes #3408
/claim #3408

Benchmark Results

Old:
[info] Benchmark                                   Mode  Cnt      Score       Error  Units
[info] RoutesBenchmark.benchmarkSmallDataZioApi   thrpt    3  32930.757 ± 20407.643  ops/s
[info] RoutesBenchmark.benchmarkSmallDataZioApi2  thrpt    3  42425.839 ±  7626.782  ops/s
[info] RoutesBenchmark.benchmarkSmallDataZioApi3  thrpt    3  51836.349 ±  2721.059  ops/s
[info] RoutesBenchmark.notFound1                  thrpt    3  22016.697 ±  2269.065  ops/s
[info] RoutesBenchmark.notFound2                  thrpt    3  19405.132 ±  2278.351  ops/s

[info] Benchmark                                            Mode  Cnt   Score    Error  Units
[info] RoundtripBenchmark.roundtripBenchmarkJavaClientAPI  thrpt    3  13.135 ± 10.142  ops/s

New:
[info] Benchmark                                   Mode  Cnt      Score      Error  Units
[info] RoutesBenchmark.benchmarkSmallDataZioApi   thrpt    3  39949.578 ±  612.926  ops/s
[info] RoutesBenchmark.benchmarkSmallDataZioApi2  thrpt    3  64732.621 ± 4345.785  ops/s
[info] RoutesBenchmark.benchmarkSmallDataZioApi3  thrpt    3  69094.709 ± 4062.645  ops/s
[info] RoutesBenchmark.notFound1                  thrpt    3  13494.307 ±  581.835  ops/s
[info] RoutesBenchmark.notFound2                  thrpt    3  21118.038 ±  423.684  ops/s

[info] Benchmark                                            Mode  Cnt   Score   Error  Units
[info] RoundtripBenchmark.roundtripBenchmarkJavaClientAPI  thrpt    3  16.851 ± 0.611  ops/s

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.

Simplify routes tree structure
1 participant