Skip to content

Make it easier to dig into large profile results#2

Open
ianawilson wants to merge 1 commit intoiddl:masterfrom
ianawilson:digging-in
Open

Make it easier to dig into large profile results#2
ianawilson wants to merge 1 commit intoiddl:masterfrom
ianawilson:digging-in

Conversation

@ianawilson
Copy link

@ianawilson ianawilson commented Dec 28, 2021

Hi, I know this project isn't actively maintained, but I found it to be a really useful base and extended it for a project I was working on to dig into performance problems in our elasticsearch cluster.

We had some complex queries and aggregations, so I needed more options to focus and prune the output. The following is all of the additions that I made. I believe all of these changes are backward compatible with any existing uses of the script.

If there is a subset of these that you'd consider including, let me know, and I'd be happy to talk through them.

Additions

  • Highlighting for operations which take longer than a given threshold (default 1000 milliseconds)
  • Option to exclude operations which are faster than a given threshold (default disabled)
  • Option to limit the depth of children to display
  • 2 levels of verbosity

Fixes

  • Aggregations were never displayed because of a variable naming conflict ("s" for "shards" and "searches")
  • All timings correctly converted from nanos to millis

- Highlighting for operations which take longer than a given threshold
  (default 1000 milliseconds)
- Option to exclude operations which are faster than a given threshold
  (default disabled)
- Option to limit the depth of children to display
- 2 levels of verbosity

Fixes

- Aggregations were never displayed because of a variable naming
  conflict ("s" for "shards" and "searches")
- All timings correctly converted from nanos to millis
@ianawilson
Copy link
Author

Hey @iddl, I wanted to tag you on this, but am not able to add any reviewers. I wanted to see if you'd be open to merging some or all of this; I found it quite helpful in digging through some big ES profiles recently on an AWS-hosted ES, where I can't get the x-pack profiler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant