Skip to content

Replace WeakConcurrentMaps with SpanConcurrentHashMaps in simple plugins #2093

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Sep 13, 2021
Merged

Replace WeakConcurrentMaps with SpanConcurrentHashMaps in simple plugins #2093

merged 6 commits into from
Sep 13, 2021

Conversation

@eyalkoren
Copy link
Contributor

@eyalkoren eyalkoren commented Aug 29, 2021 •
edited by zube bot
Loading

Implementing #2092 for several plugins where replacement is straightforward:

  • AsyncHttpClient
  • Dubbo
  • Lettuce
  • External process tracing

@eyalkoren eyalkoren mentioned this pull request Aug 29, 2021
Closed
9 tasks
@ghost
Copy link

ghost commented Aug 29, 2021 •
edited by ghost
Loading

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-09-13T06:10:46.651+0000

  • Duration: 56 min 0 sec

  • Commit: 5405a4d

Test stats 🧪

Test Results
Failed 0
Passed 2396
Skipped 19
Total 2415

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 2396
Skipped 19
Total 2415

@SylvainJuge SylvainJuge self-requested a review August 30, 2021 08:06
@AlexanderWert AlexanderWert added this to the 7.16 milestone Aug 30, 2021
@eyalkoren eyalkoren force-pushed the switch-to-SpanConcurrentHashMap-2 branch from 588a332 to 5405a4d Compare September 13, 2021 06:10
@eyalkoren eyalkoren merged commit 2533c5e into elastic:master Sep 13, 2021
@eyalkoren eyalkoren deleted the switch-to-SpanConcurrentHashMap-2 branch September 13, 2021 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SylvainJuge SylvainJuge SylvainJuge approved these changes

Assignees

@eyalkoren eyalkoren

Labels

agent-java chore

Projects

None yet

Milestone

7.16

Development

Successfully merging this pull request may close these issues.

3 participants

@eyalkoren @SylvainJuge @AlexanderWert