microsoft / testfx

MSTest framework and adapter

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Use `ConcurrentDictionary` for attribute cache

nohwnd opened this issue · comments

Evangelink 2 days ago
What about using ConcurrentDictionary instead?

Member
Author
@nohwnd nohwnd 2 days ago
I was thinking about it I wanted to measure the impact once this is merged, I saw time spent in locks in PerfView, but it also was not significantly faster without any locks. So please merge this approach first.

Member
Author
@nohwnd nohwnd 26 minutes ago
I was also thinking about writer only lock, but I am not sure if concurrent dictionary does not do that for me automatically already.