Support profiling CPU + Wall clock together

grafana / pyroscope-java

pyroscope java integration

Apache License 2.0

72 stars 31 forks source link

Support profiling CPU + Wall clock together #138

Open be-hase opened 5 months ago

be-hase commented 5 months ago

It seems to be possible to get both since async profiler v3. https://github.com/async-profiler/async-profiler/issues/740

How about also supporting pyroscope?

korniltsev commented 5 months ago

We will look into it.

In the meantime it is already possible to have wall + cpu by using wall. The pyroscope assigns events STATE_RUNNABLE to extra process_cpu profile type.

korniltsev commented 5 months ago

We also do not recommend wall for continuous profiling because of bigger overhead.