-
Notifications
You must be signed in to change notification settings - Fork 372
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
DEBUG-2657 prefix benchmark names with the product as per their source file #3828
Conversation
1d08d50
to
52967e5
Compare
benchmarks/benchmarks_helper.rb
Outdated
module JobReporter | ||
def report(name, *args, **opts, &block) | ||
caller_path = caller_locations.first.path | ||
prefix = File.basename(caller_path).sub(/_.*\z/, '') | ||
super("#{prefix} - #{name}", *args, **opts, &block) | ||
end | ||
end | ||
|
||
class Benchmark::IPS::Job | ||
prepend JobReporter | ||
end |
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.
This seems to be breaking (older?) versions of benchmark/ips (on older Rubies)
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.
👍 LGTM
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3828 +/- ##
=======================================
Coverage 97.83% 97.83%
=======================================
Files 1264 1264
Lines 75678 75678
Branches 3720 3720
=======================================
+ Hits 74039 74042 +3
+ Misses 1639 1636 -3 ☔ View full report in Codecov by Sentry. |
bbfe1cb
to
d8b13dd
Compare
d8b13dd
to
b37f157
Compare
* master: Suppress startup logs in test and development environments [PROF-10241] Fix issue in tests by simplifying `at_fork` monkey patch Minor: Check pid before hash Rename ProcessDaemonMonkeyPatch -> ProcessMonkeyPatch [PROF-10241] Use Process._fork hook in `at_fork` monkey patch on Ruby 3.1+ Minor: Bootstrap empty rbs file Rubocop fixes Update Steepfile Add note about not re-registering the same block [PROF-10241] Extract profiler `at_fork` monkey patch to utils Remove typechecking skeleton Rename `Kernel` monkey patch to `KernelMonkeyPatch` to avoid confusion Rename `at_fork` -> `datadog_at_fork` to avoid any clashes Minor: Avoid returning process after monkey patching Rename ProcessDaemonPatch -> ProcessDaemonMonkeyPatch Clean up test skipping to not depend on profiling Minor: Clean up storage initialization + don't leak storage Quick pass at cleaning up/refreshing comments
* master: (31 commits) DEBUG-2657 prefix benchmark names with the product as per their source file (DataDog#3828) [NO-TICKET] Don't put results in benchmarking folder directly Suppress startup logs in test and development environments [PROF-10241] Fix issue in tests by simplifying `at_fork` monkey patch Minor: Check pid before hash Rename ProcessDaemonMonkeyPatch -> ProcessMonkeyPatch [PROF-10241] Use Process._fork hook in `at_fork` monkey patch on Ruby 3.1+ Minor: Bootstrap empty rbs file Rubocop fixes Update Steepfile Add note about not re-registering the same block [PROF-10241] Extract profiler `at_fork` monkey patch to utils Remove typechecking skeleton Rename `Kernel` monkey patch to `KernelMonkeyPatch` to avoid confusion Rename `at_fork` -> `datadog_at_fork` to avoid any clashes Minor: Avoid returning process after monkey patching Rename ProcessDaemonPatch -> ProcessDaemonMonkeyPatch Clean up test skipping to not depend on profiling Minor: Clean up storage initialization + don't leak storage Quick pass at cleaning up/refreshing comments ...
What does this PR do?
Prefixes each benchmark name with the product prefix.
Motivation:
At 37 current benchmarks, their list is awkward to look through. The prefix will make benchmarks grouped by product which will make them easier to read.
Additional Notes:
How to test the change?
Benchmark platform UI
Unsure? Have a question? Request a review!