Skip to content
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

Split single step instrument entry point #3931

Merged
merged 4 commits into from
Sep 20, 2024
Merged

Conversation

TonyCTHsu
Copy link
Contributor

@TonyCTHsu TonyCTHsu commented Sep 20, 2024

What does this PR do?

For single step instrumentation, use a different entry point that would rescue LoadError and not crash the process.

@pr-commenter
Copy link

pr-commenter bot commented Sep 20, 2024

Benchmarks

Benchmark execution time: 2024-09-20 17:05:11

Comparing candidate commit 8b71df7 in PR branch tonycthsu/ssi-entry-point with baseline commit e163c21 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 23 metrics, 2 unstable metrics.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.75%. Comparing base (9b1d33b) to head (f69aac5).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3931      +/-   ##
==========================================
- Coverage   97.77%   97.75%   -0.02%     
==========================================
  Files        1297     1297              
  Lines       77611    77611              
  Branches     3871     3871              
==========================================
- Hits        75881    75868      -13     
- Misses       1730     1743      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@TonyCTHsu TonyCTHsu marked this pull request as ready for review September 20, 2024 16:42
@TonyCTHsu TonyCTHsu requested review from a team as code owners September 20, 2024 16:42
@TonyCTHsu TonyCTHsu added single-step Single Step APM Instrumentation dev/internal Other internal work that does not need to be included in the changelog labels Sep 20, 2024
lib/datadog/single_step_instrument.rb Outdated Show resolved Hide resolved
@TonyCTHsu TonyCTHsu merged commit 4eeda1b into master Sep 20, 2024
159 of 170 checks passed
@TonyCTHsu TonyCTHsu deleted the tonycthsu/ssi-entry-point branch September 20, 2024 17:48
@github-actions github-actions bot added this to the 2.4.0 milestone Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/internal Other internal work that does not need to be included in the changelog single-step Single Step APM Instrumentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants