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

Reduce CPU usage on access to java.util.HashTable #327

Conversation

plokhotnyuk
Copy link
Contributor

@plokhotnyuk plokhotnyuk commented Jul 20, 2022

Fixing of overlooked capacity sizing. Below are results of benchmarks for real-world messages on JDK 17.

Before

[info] Benchmark                                  (size)   Mode  Cnt         Score         Error  Units
[info] GoogleMapsAPIReading.smithy4sJson             N/A  thrpt    5     15125.336 ±      34.299  ops/s
[info] OpenRTBReading.smithy4sJson                   N/A  thrpt    5    177589.601 ±    2958.052  ops/s
[info] TwitterAPIReading.smithy4sJson                N/A  thrpt    5     34670.625 ±    1334.928  ops/s

[info] Benchmark                                  (size)   Mode  Cnt         Score         Error  Units
[info] GoogleMapsAPIReading.smithy4sJson             N/A  thrpt    5     16034.257 ±      40.094  ops/s
[info] OpenRTBReading.smithy4sJson                   N/A  thrpt    5    183281.757 ±    7169.430  ops/s
[info] TwitterAPIReading.smithy4sJson                N/A  thrpt    5     36217.351 ±     284.032  ops/s

@plokhotnyuk plokhotnyuk force-pushed the reduce-cpu-usage-on-access-to-hashtables branch from 7be52f9 to 64d43f9 Compare July 20, 2022 13:56
@plokhotnyuk plokhotnyuk force-pushed the reduce-cpu-usage-on-access-to-hashtables branch from 64d43f9 to 77e3a52 Compare July 20, 2022 14:48
@Baccata Baccata merged commit 786d386 into disneystreaming:main Jul 20, 2022
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.

None yet

2 participants