-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Closed
Labels
area-Extensions-Optionshelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorstenet-performancePerformance related issuePerformance related issue
Milestone
Description
Description
Today both IOptions<T> and IOptionsSnapshot<T> share an implementation OptionsManager<T>. This implements both IOptions<T> and IOptionsSnapshot<T> and ends up paying the cost of the concurrent dictionary for IOptions<T> when the value is set once and never changes.
We can split the implementation into a different type that implements IOptions<T> and doesn't use the cache.
stephentoub and strangeman375
Metadata
Metadata
Assignees
Labels
area-Extensions-Optionshelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorstenet-performancePerformance related issuePerformance related issue