-
Notifications
You must be signed in to change notification settings - Fork 322
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
PROF-8714: GA Code hotspots and endpoint collection by defaulting them to true #3940
Conversation
Overall package sizeSelf size: 5.83 MB Dependency sizes
🤖 This report was automatically generated by heaviest-objects-in-the-universe |
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.
Nice !! 🎉
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3940 +/- ##
==========================================
+ Coverage 84.69% 85.01% +0.31%
==========================================
Files 238 238
Lines 10176 10176
Branches 33 33
==========================================
+ Hits 8619 8651 +32
+ Misses 1557 1525 -32 ☔ View full report in Codecov by Sentry. |
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.
I think you need to update the config test to reflect the new defaults.
3aa03a8
to
65a96d3
Compare
65a96d3
to
d0c2b92
Compare
BenchmarksBenchmark execution time: 2024-01-08 14:59:23 Comparing candidate commit d0c2b92 in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 259 metrics, 6 unstable metrics. scenario:plugin-graphql-with-depth-off-18
|
What does this PR do?
Changes the defaults for code hotspots and endpoint collection to true.
Motivation
Making these features GA.
Additional Notes
🎉
Security
Datadog employees:
@DataDog/security-design-and-guidance
.