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.

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

Update partial transaction failure handling and add a test #1818

Merged
merged 2 commits into from
May 9, 2023
Merged

Update partial transaction failure handling and add a test #1818

merged 2 commits into from
May 9, 2023

Conversation

basepi
Copy link
Contributor

@basepi basepi commented Apr 28, 2023

What does this pull request do?

Updates the failure behavior for partial transaction registering to match the NodeJS agent, as described here

Related issues

Ref elastic/apm#753
Ref #1784

@basepi basepi added this to the 8.9 milestone Apr 28, 2023
@basepi basepi requested a review from beniwohli April 28, 2023 19:43
@basepi basepi self-assigned this Apr 28, 2023
@basepi basepi merged commit 82603f0 into elastic:main May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@beniwohli beniwohli beniwohli approved these changes

Assignees

@basepi basepi

Labels
agent-python
Projects
None yet
Milestone
8.9
Development

Successfully merging this pull request may close these issues.
2 participants
@basepi @beniwohli