Skip to content

Conversation

jackshirazi
Copy link
Contributor

What does this PR do?

Correct SQS transaction lifetime for concurrent flow message processing. The react flow interleaves messages to different receivers and this was not being correctly handled

Checklist

  • This is an enhancement of existing features, or a new feature in existing plugins
    • I have updated CHANGELOG.next-release.md
    • 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
  • This is a bugfix
  • This is a new plugin
    • I have updated CHANGELOG.next-release.md
    • My code follows the style guidelines of this project
    • I have made corresponding changes to the documentation
    • I have added tests that prove my fix is effective or that my feature works
    • New and existing unit tests pass locally with my changes
    • I have updated supported-technologies.md
    • Added an API method or config option? Document in which version this will be introduced
    • Added an instrumentation plugin? Describe how you made sure that old, non-supported versions are not instrumented by accident.
  • This is something else

Copy link

github-actions bot commented Sep 3, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@jackshirazi jackshirazi changed the title Adjust transaction lifetime to handle flux flow Adjust SQS message transaction lifetime to handle flux flow Sep 3, 2025
@jackshirazi jackshirazi marked this pull request as ready for review September 10, 2025 11:05
@jackshirazi jackshirazi requested a review from a team as a code owner September 10, 2025 11:05
…/co/elastic/apm/agent/awssdk/common/AbstractMessageIteratorWrapper.java

Co-authored-by: SylvainJuge <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants