Skip to content

feat: add experimental profiler #528

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

Merged
merged 9 commits into from
Mar 18, 2025
Merged

feat: add experimental profiler #528

merged 9 commits into from
Mar 18, 2025

Conversation

maxday
Copy link
Contributor

@maxday maxday commented Mar 18, 2025

Description of changes:
adds AWS Lambda Profiler Extension for Java

The Lambda profiler extension allows you to profile your Java functions invoke by invoke, with high fidelity, and no
code changes. It uses the async-profiler project to produce
profiling data and automatically uploads the data as HTML flame graphs to S3.

Target (OCI, Managed Runtime, both):
both

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

msailes and others added 5 commits March 18, 2025 10:16
* fix: role name for integration tests

* fix: role name for integration tests

* fix: role name for integration tests

* fix: role name for integration tests
Added information regarding the alpha status of the release.
* Profiler updates

Moved profiler to 'experimental' folder
Added AWS_LAMBDA_ prefix to env vars
Updated README.md

* fix: IAM permissions, wrapper, port communication and integration test

---------

Co-authored-by: Maxime David <[email protected]>
* Update profiler README.md

Update license details in profiler README.md

* Fix profiler links on main README

Add /experimental/ path for profiler links on main README

* README tweaks
Copy link
Collaborator

@msailes msailes left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Collaborator

@krk krk left a comment

Choose a reason for hiding this comment

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

lgtm

@maxday maxday merged commit 574ac85 into main Mar 18, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants