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

BDN for parsing + ACL #446

Merged
merged 3 commits into from
Jun 6, 2024
Merged

BDN for parsing + ACL #446

merged 3 commits into from
Jun 6, 2024

Conversation

badrishc
Copy link
Contributor

@badrishc badrishc commented Jun 6, 2024

Uses a batch size of 128 commands per BDN call invocation.
Also: change GET/SET to not clobber input buffer in order to work correctly with BDN + it is a cleaner design.

@badrishc badrishc mentioned this pull request Jun 6, 2024
9 tasks
@badrishc
Copy link
Contributor Author

badrishc commented Jun 6, 2024

BenchmarkDotNet v0.13.12, Windows 10 (10.0.19045.4412/22H2/2022Update)
Intel Xeon CPU E5-2690 v4 2.60GHz, 2 CPU, 56 logical and 28 physical cores
.NET SDK 8.0.300
  [Host] : .NET 8.0.6 (8.0.624.26715), X64 RyuJIT AVX2
  .NET 6 : .NET 6.0.31 (6.0.3124.26714), X64 RyuJIT AVX2
  .NET 8 : .NET 8.0.6 (8.0.624.26715), X64 RyuJIT AVX2

Server=True
Method Job EnvironmentVariables Runtime Mean Error StdDev Allocated
InlinePing .NET 6 Empty .NET 6.0 4.071 us 0.0563 us 0.0499 us -
Set .NET 6 Empty .NET 6.0 34.649 us 0.5962 us 0.5577 us -
Get .NET 6 Empty .NET 6.0 20.259 us 0.1348 us 0.1195 us -
InlinePing .NET 8 DOTNET_TieredPGO=0 .NET 8.0 2.560 us 0.0078 us 0.0065 us -
Set .NET 8 DOTNET_TieredPGO=0 .NET 8.0 25.828 us 0.2689 us 0.2515 us -
Get .NET 8 DOTNET_TieredPGO=0 .NET 8.0 15.637 us 0.1448 us 0.1355 us -

@badrishc badrishc merged commit eff8249 into main Jun 6, 2024
26 checks passed
@badrishc badrishc deleted the badrishc/resp-parse-stress branch June 6, 2024 16:00
@github-actions github-actions bot locked and limited conversation to collaborators Aug 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant