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

Add support for partial transaction reporting in Lambda #1784

Merged
merged 16 commits into from Apr 19, 2023

Conversation

basepi
Copy link
Contributor

@basepi basepi commented Mar 27, 2023

What does this pull request do?

The lambda extension now has support for early reporting of partial transactions. This covers the case where our lambda runtime times out before we can properly end and report the transaction. In this case, the lambda extension will report the transaction with a timeout result.

Related issues

Closes #1734

@basepi basepi requested a review from beniwohli April 18, 2023 19:50
@basepi basepi marked this pull request as ready for review April 18, 2023 19:50
Copy link
Contributor

@beniwohli beniwohli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some minor comments. Great work!

elasticapm/transport/base.py Show resolved Hide resolved
elasticapm/traces.py Show resolved Hide resolved
elasticapm/transport/http.py Outdated Show resolved Hide resolved
@basepi basepi merged commit 25eb517 into elastic:main Apr 19, 2023
95 checks passed
APM-Agents (OLD) automation moved this from In Progress to Done Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

[META 753] Support Partial Transaction for AWS Lambda
2 participants