Oh, I have opinions all right…
We’re already rocking about 60 cargo deps - so there’s that 😆
I haven’t dug into that specific library, but a couple thoughts:
• Does that have any impact on runtime when it’s not in use? In my experience, profilers need to do a bunch of instrumentation, so can that be noop’d at runtime with a flag?
• Does this open us up to “what about adding Profiler XYZ”?
• Once it gets added and then exposed to a CLI, it’s basically a permanent fixture to Pants
• What set of feature flags would we ship our binary with (I’m not entirely sure if pprof’s “features” are compile time or strictly runtime)?
The pros/cons list on pprof wasn’t really convincing for me, but I do think there is a world where we have a feature gated profiling mode - which doesn’t ship with the Pants binary, but makes it easier to build from source with profilers integrated 🤷