No support to run multiple profile

Question

No support to run multiple profile

prakashettigar opened this issue 3 years ago · 6 comments

profile.Start(profile.MemProfile, profile.ProfilePath("."), profile.NoShutdownHook)
profile.Start(profile.CPUProfile, profile.ProfilePath("."), profile.NoShutdownHook)

Starting multiple profiles throws an error : profile: Start() already called

Answer 1 · 2021-08-30T20:27:31.000Z

This is a deliberate design choice.

Answer 2 · 2022-05-09T15:14:04.000Z

@davecheney can you please explain why?

Answer 3 · 2022-05-09T16:11:36.000Z

Running multiple profiles at the same time causes interference.

Answer 4 · 2022-10-19T13:42:25.000Z

@davecheney is the interference something that breaks or just skew results? For example with mem+cpu the coarse relative picture seems to hold up well in cases I've checked despite the overhead.

Answer 5 · 2022-10-19T18:39:09.000Z

It skews the results.

Answer 6 · 2023-11-16T09:27:12.000Z

I suggest to add the explanation to the actual error, as it's not trivial as in fact it makes the pkg do more than just a convenient proxy to the native API, because the native API does allow running multiple profilers.

If you agree I can make the PR