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

Use FrozenDictionary in Router #3698

Merged

bernardnormier merged 1 commit into icerpc:main from bernardnormier:frozen-dictionary

Oct 18, 2023

Member

bernardnormier commented Oct 17, 2023

According to https://devblogs.microsoft.com/dotnet/performance-improvements-in-aspnet-core-8/, Frozen dictionary is significantly faster for the "router" use-case.


          Use FrozenDictionary in Router

1ba075f

bernardnormier requested review from pepone and InsertCreativityHere

October 17, 2023 20:24

InsertCreativityHere approved these changes

View reviewed changes

Member

InsertCreativityHere left a comment

Seems pretty cool. (get it?)

I wonder how much a performance difference it will make for us.

pepone approved these changes

View reviewed changes

bernardnormier merged commit 94a7662 into icerpc:main

5 checks passed

bernardnormier deleted the frozen-dictionary branch

October 19, 2023 20:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet