-
Notifications
You must be signed in to change notification settings - Fork 442
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[O11y][MySQL] Rally benchmark mysql.slowlog
#9024
Conversation
mysql.slowlog
🚀 Benchmarks reportTo see the full report comment with |
fields: | ||
- name: timestamp | ||
period: -24h # one day | ||
- name: start_time |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this mean start time could be after end time? also, should timestamp and start_time be the same?
it would be good if we could generate durations, so we could have the start time, then say we want end times between 100-1000 ms later, for example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @tommyers-elastic for this catch. @aspacca also thanks for your suggestion link.
If we are giving multiple date fields the timestamps would be same which would not make sense. Hence changed the start_time
to before 5m
and end_time
to before 2m
the timestamp generated. Let me know if this makes sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perfect, thanks! if these aren't documented we should add something.
only one more comment (not a blocker), could we make these have some variation?
{{- $startTime := generate "timestamp" | date_modify "-5m" }}
{{- $endTime := generate "timestamp" | date_modify "-2m" }}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tommyers-elastic Agree. Generated some range of enum values which the start and end time can take.
The date_modify
is documented here as a part of this documentation. Is that what you were referring to?
💚 Build Succeeded
History
cc @aliabbas-elastic |
|
@tommyers-elastic if all good can we go ahead merging this PR. Let me know if there are any comments. Thanks !! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on the other comments I see in the PR; approving this.
Proposed commit message
slowlog
data stream ofMySQL
Checklist
How to test this PR locally
Run this command from package root
elastic-package benchmark rally --benchmark slowlog-benchmark -v
elastic-package benchmark stream --benchmark slowlog-benchmark -v
Related issues
Screenshots