feat(debug): add more metrics in ThreadDebugState#2282
feat(debug): add more metrics in ThreadDebugState#2282alexandre-daubois wants to merge 1 commit intophp:mainfrom
Conversation
3968c59 to
aae0bd5
Compare
|
Could we measure the performance cost of this? Maybe should this feature be put behind a flag? |
|
Did I just spend 10 hours attempting to figure out why my other branch segfaults on 8.2 debian images and it's not actually related to my branch in the end? Certainly seems like it... |
I'll do benchmarks soon and we'll be able to see from there! |
|
This is the impact, on already existing benchmark:
Also, 0 additional allocation is recorded on the hot path. The impact is negligible and the metrics newly exposed would really help better monitor how things are going while processing requests. |
Sounds reasonable to me. Out of curiosity, does anybody know of a per-route monitoring solution that exposes metrics like number of requests, php request duration, etc.? Much like Sentry, Datadog or Tideways, but without all the low-level insights? Or perhaps that's a good candidate for a go php extension to hook into the runtime, gather the data, hand it to Go and add it to the prometheus metrics? |
|
Not that I'm aware of, but an extension for this purpose could indeed be a good idea |
I'm working on an observability tool and exposing those metrics would be useful. For each thread, it exposes the current URI being processed, the method, the request start date, the request count and the memory usage.