ETW events should use default '/' instead of '_'

[DevPaulLiu]

NuGet Product Used

Visual Studio Package Manager Console

Product Version

N/A

Worked before?

N/A

Impact

None

Repro Steps & Context

There is an issue in performance tests, not sure if this impact any of NuGet telemetry monitors.

As per David Kean:
We should avoid converting event/property names in ETW provider
This is allocating a lot during solution close and other places where lots of events are being produced, which is causing GCs to occur in some runs, introducing noise. The statement that says ETW events can't have '/' seems to be incorrect, as Windows, VS & .NET all include events with '/'. Also stopped converting properties as these are raw strings.

Verbose Logs

No response

[nkolev92]

The issue was not referenced via a fixes statement so it wasn't closed. I've amended the PR body.

Closing this as completed.