Skip to content

WIP: dump query processing performance metrics from various stages#3371

Closed
himanshug wants to merge 5 commits intoapache:masterfrom
himanshug:dump_perf
Closed

WIP: dump query processing performance metrics from various stages#3371
himanshug wants to merge 5 commits intoapache:masterfrom
himanshug:dump_perf

Conversation

@himanshug
Copy link
Copy Markdown
Contributor

Depends on #3319 and #3323 and reviewable only after they are merged.

Fixes #3324

it can be enabled by sending "dumpQueryPerf" attribute in query context, value is expected to be an integer.
1 means only dump stats from broker
1+ means include stats from historicals too , bigger number means more segment's stats will be reported.

@himanshug himanshug added this to the 0.9.3 milestone Aug 17, 2016
@fjy
Copy link
Copy Markdown
Contributor

fjy commented Nov 8, 2016

is this still WIP?

@himanshug
Copy link
Copy Markdown
Contributor Author

depends on #3319 and #3323 , so removing it from 0.9.3 till those are merged

@himanshug himanshug removed this from the 0.9.3 milestone Nov 9, 2016
@stale
Copy link
Copy Markdown

stale Bot commented Feb 28, 2019

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@druid.apache.org list. Thank you for your contributions.

@stale stale Bot added the stale label Feb 28, 2019
@stale
Copy link
Copy Markdown

stale Bot commented Mar 7, 2019

This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

@stale stale Bot closed this Mar 7, 2019
seoeun25 pushed a commit to seoeun25/incubator-druid that referenced this pull request Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants