-
Notifications
You must be signed in to change notification settings - Fork 17
Open
Description
... as we have a setter now and already using it
Lines 69 to 75 in e6bf3b4
| // Once we have public Activity.Current setter (https://github.com/dotnet/corefx/issues/29207) this method will be | |
| // simplified to just assign Current if is was lost. | |
| // In the mean time, we are creating child Activity to restore the context. We have to send | |
| // event with this Activity to tracing system. It created a lot of issues for listeners as | |
| // we may potentially have a lot of them for different stages. | |
| // To reduce amount of events, we only care about ExecuteRequestHandler stage - restore activity here and | |
| // stop/report it to tracing system in EndRequest. |
Metadata
Metadata
Assignees
Labels
No labels