8000 Async can now wait on materialized objects by bbakerman · Pull Request #3462 · graphql-java/graphql-java · GitHub
[go: up one dir, main page]

Skip to content

Async can now wait on materialized objects#3462

Closed
bbakerman wants to merge 3 commits intomasterfrom
async-on-objects
Closed

Async can now wait on materialized objects#3462
bbakerman wants to merge 3 commits intomasterfrom
async-on-objects

Conversation

@bbakerman
Copy link
Member

This is a setup PR for later work. This allows Async to wait on materialised objects or CFs or a mix

This wont be used in the code today - all values will be CFs but this is setup work for the future

@bbakerman
Copy link
Member Author
bbakerman commented Feb 23, 2024

after update with batchSize=100 - agained edited these numbers

before

Benchmark                      (numberOfFieldCFs)   Mode  Cnt       Score       Error  Units
AsyncBenchmark.benchmarkAsync                   1  thrpt    4  687152.037 ± 819652.858  ops/s
AsyncBenchmark.benchmarkAsync                   5  thrpt    4  158146.382 ±  13373.615  ops/s
AsyncBenchmark.benchmarkAsync                  20  thrpt    4   28521.619 ±   2698.366  ops/s

after

Benchmark                      (numberOfFieldCFs)   Mode  Cnt       Score       Error  Units
AsyncBenchmark.benchmarkAsync                   1  thrpt    4  580070.646 ± 83754.940  ops/s
AsyncBenchmark.benchmarkAsync                   5  thrpt    4  133897.900 ±  7252.349  ops/s
AsyncBenchmark.benchmarkAsync                  20  thrpt    4   28299.556 ± 28528.998  ops/s

@bbakerman
Copy link
Member Author

No longer needed - the co-dependent PRs brough in this code

@bbakerman bbakerman closed this Mar 27, 2024
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.

2 participants

0