You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As part of the Fortunes optimizations, it was discovered that our context pooling maximum size wasn't high enough (aspnet/Benchmarks#1630). As suggested by @sebastienros, we should have a logging event for this - it's very easy to miss this (as we did) and think you're efficiently using context pooling, when in fact you're not.
IMHO it makes sense to fire this event only once, as a warning by default. Especially zealous applications (like our benchmarks) could configure this to throw an exception instead.
The text was updated successfully, but these errors were encountered:
As part of the Fortunes optimizations, it was discovered that our context pooling maximum size wasn't high enough (aspnet/Benchmarks#1630). As suggested by @sebastienros, we should have a logging event for this - it's very easy to miss this (as we did) and think you're efficiently using context pooling, when in fact you're not.
IMHO it makes sense to fire this event only once, as a warning by default. Especially zealous applications (like our benchmarks) could configure this to throw an exception instead.
The text was updated successfully, but these errors were encountered: