Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need better process information caching #523

Open
jander-msft opened this issue Jul 1, 2021 · 0 comments
Open

Need better process information caching #523

jander-msft opened this issue Jul 1, 2021 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@jander-msft
Copy link
Member

In 'connect' mode, nearly all of the process information for every discoverable process is recomputed for every http call. This is very inefficient and can lead to inconsistency of data between calls. For example, one the tests makes slightly different /process calls and validates that the identifier information is consistent between calls. The results from one of these runs is:

- PID:  2449
  UID:  00000000-0000-0000-0000-000000000000
  Name: dotnet
- PID:  2449
  UID:  00000000-0000-0000-0000-000000000000
  Name: unknown
- PID:  2449
  UID:  00000000-0000-0000-0000-000000000000
  Name: dotnet

Note that the name is "unknown" for the second call, but was successfully computed for the prior and subsequent calls.

The tool should try to (1) for process information that requires "lengthy" computation (issuing ProcessInfo command, use event source trace to get command line information, etc), allow those computations to continue but stop waiting for their result for the current request after a short period of time, and (2) cache any successfully computed information for aspects that will not change over the lifetime of the process.

@jander-msft jander-msft added this to the GA milestone Jul 1, 2021
@jander-msft jander-msft added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Jul 1, 2021
@jander-msft jander-msft changed the title Add better process information caching Need better process information caching Jul 1, 2021
@jander-msft jander-msft modified the milestones: 6.0.0, Future Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant