Paul Harrington

PerfView uses [MS DIA](https://learn.microsoft.com/visualstudio/debugger/debug-interface-access/debug-interface-access-sdk) to load (native) symbols, so the sharing mode isn't under PerfView's direct control. It sounds like you're trying to delete the PDB while PerfView is still...

The nuget is built after the build of TraceEvent itself. TraceEvent.csproj sets `GeneratePackageOnBuild` to `true. The .nuspec's placeholder properties, (Configuration, version and OutDir) are set in a custom target called...

Is there more to this PR? I mean, I see where TimeStampUtc, is introduced which doesn't include the local timezone conversion, but where is this new property being consumed? The...

I've no concerns. Once this PR is in, you could consider: 1. Deprecating the non-UTC properties via [Obsolete] and steering people to use the UTC versions. 2. Auditing all the...

You need to tell Kestrel about the certificate too. ```csharp .ConfigureKestrel((context, opt) => { opt.ConfigureHttpsDefaults(adapterOptions => { adapterOptions.ServerCertificate = /* TODO: Load your Cert */; }); }) ``` You'll need...

There isn't an option to disable the telemetry. > Should there be one added so that the user can opt-out? Well, you tell me. Should there be one? And if...

OK. How should the opportunity to opt-out be exposed? Perhaps a Tools/Options... page.

> This issue must be fixed Thanks for bringing that up, @sharwell. Can you refer me to documentation or a license agreement? While I work for Microsoft, the Editor Guidelines...

The trouble I have with using the privacy settings dialog is that it refers to a privacy policy that talks about Microsoft's data collection and the way telemetry is used...

Any further comments? If not, can I get a review and sign-off?