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

Enhance/document msg listener heuristic #3299

Merged
merged 10 commits into from
Aug 25, 2023

Conversation

SylvainJuge
Copy link
Member

@SylvainJuge SylvainJuge commented Aug 23, 2023

What does this PR do?

Fixes #3135
Relates to #3259

I had to move the test JMS classes to a package outside of co.elastic.apm as co.elastic.apm is already covered by the test configuration which is automatically loaded from classpath.

Checklist

  • This is an enhancement of existing features, or a new feature in existing plugins
    • I have updated CHANGELOG.asciidoc
    • I have added tests that prove my fix is effective or that my feature works
    • Added an API method or config option? Document in which version this will be introduced
    • I have made corresponding changes to the documentation

@SylvainJuge SylvainJuge marked this pull request as ready for review August 24, 2023 09:41
@github-actions
Copy link

/test

@SylvainJuge SylvainJuge requested a review from a team August 24, 2023 09:41
…figuration/MessagingConfiguration.java

Co-authored-by: jackshirazi <[email protected]>
…figuration/MessagingConfiguration.java

Co-authored-by: jackshirazi <[email protected]>
@SylvainJuge SylvainJuge enabled auto-merge (squash) August 25, 2023 15:16
@SylvainJuge SylvainJuge merged commit b752467 into elastic:main Aug 25, 2023
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Allow opt-out for MessageListener matcher performance optimization
2 participants