microsoft / microsoft/Power-Fx
Add telemetry helper
Open
@LucGenetier is already working on this.
Since Mar 2, 2023.
- Dominant language
- C#
- Stars
- 3.4k
- Forks
- 358
- Avg merge
- 10h 34m
- Merged PRs (30d)
- 3
Description
For Check, Eval add a helper that accepts ILogger and captures (As appropriate):
- exceptions thrown - ToString(), type, callstack
- Expression Length
- Anonymized Expression (see https://github.com/microsoft/Power-Fx/blob/9178b974eced29fd06030d0f3ec563fa0048cf74/src/libraries/Microsoft.PowerFx.Core/Public/CheckResult.cs#L519 ). Probably allow host to customize this.
- Duration MS of check
- Duration MS of Eval
Some optional additional information provided by host:
- anonymized user
- "document id"
Also could apply this to LSP. Notably, we want:
- duration
- message kind
- any exceptions thrown
See Fx test site for examples.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.