Add Query Monitor support

humanmade / aws-xray

HM Platform AWS X-Ray Integration

23 stars 4 forks source link

Add Query Monitor support #36

Closed joehoyle closed 5 years ago

joehoyle commented 5 years ago

Working on X-Ray locally can be quite tough, as it requires either hooking it up to a live AWS account, or manually printing out data that would be sent to the daemon. This adds a new Query Monitor panel for all the segments sent to AWS X-Ray for inspection. It also adds the Flamegraph for the request in a new panel, so it can be used as a profiling tool too.

Screenshots:

joehoyle commented 5 years ago

[x] Need to only enqueue scripts when Query Monitor is actually enabled

rmccue commented 5 years ago

Does this affect page load times? I suspect this is a lot of data.

joehoyle commented 5 years ago

@rmccue not really, this is all data that is already processed, so it's just a case of outputting it in the Outputter, so more page weight, though I don't think as much as say, the rest of the query monitor output.

joehoyle commented 5 years ago

Thanks rob, will get on this!

joehoyle commented 5 years ago

Cheers @roborourke, I added translation functions and also some more doc blocks.