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

[#11052] Fix Caffeine Executor to not use jdk common pool #11054

Merged
merged 1 commit into from
May 23, 2024

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented May 22, 2024

No description provided.

Copy link

codecov bot commented May 23, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 37.14%. Comparing base (03cc842) to head (4a5ff1d).
Report is 3 commits behind head on master.

Files Patch % Lines
...rcorp/pinpoint/profiler/cache/ExecutorManager.java 80.76% 3 Missing and 2 partials ⚠️
...rcorp/pinpoint/profiler/cache/CaffeineBuilder.java 80.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11054      +/-   ##
============================================
+ Coverage     37.12%   37.14%   +0.02%     
- Complexity    11970    11980      +10     
============================================
  Files          3929     3931       +2     
  Lines         96888    96919      +31     
  Branches      10380    10383       +3     
============================================
+ Hits          35967    35998      +31     
+ Misses        57967    57965       -2     
- Partials       2954     2956       +2     

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

@emeroad emeroad merged commit 1d37e5a into pinpoint-apm:master May 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant