Skip to content

Conversation

@M-Elsaeed
Copy link
Contributor

Lambda Managed Instances, Version Bumps, and Added Tests.
Issue #, if available: -

Description of changes: Lambda Managed Instances Implementation and Tests. RIC Version Bump to 2.8.7. Please Check Commits History.

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.

maxday and others added 26 commits December 4, 2025 15:34
…cal C++ Curl Client (#82)

Multi Concurrent RIC and Enable Curl Multi Concurrent Requests
* Merge from Public Repo

* Fix Failing Tests

---------

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
* Support Multiconcurrent Logging
---------

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
* Modified RIC Behavior for Retries and Failure Handling + Refactoring
Merge from Public to Private Repo
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
…is set (#90)

* Only Log Concurrency Warning Message when AWS_LAMBDA_MAX_CONCURRENCY is set

* Var name change

---------

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
* Version Bump RIC to 2.8.1

---------

Authored-by: Mohammed Ehab <moehabe@amazon.com>
…e and Version Bump to 2.8.2 (#92)

* Allow AWS_LAMBDA_MAX_CONCURRENCY to be one and crash the RIC Otherwise.

* Version Bump to 2.8.2

---------

Authored-by: Mohammed Ehab <moehabe@amazon.com>
… dependencies

- Bump aws-lambda-java-serialization from 1.1.5 to 1.1.6
- Bump aws-lambda-java-events from 3.16.0 to 3.16.1
- Update aws-lambda-java-tests dependencies to use new versions
- Update aws-lambda-java-runtime-interface-client serialization dependency 1.1.2→1.1.6
- Update aws-lambda-java-events-sdk-transformer events dependency 3.11.2→3.16.1
…endency

Add local build step for aws-lambda-java-serialization before building runtime interface client.

## Why This Fix is Needed

The runtime interface client depends on aws-lambda-java-serialization version 1.1.6,
but this version doesn't exist in Maven Central yet. By building and installing the
serialization package locally first, we ensure:

1. The correct version (1.1.6) is available in the local Maven repository
2. The runtime interface client build won't fail looking for a non-existent version on Maven Central
3. The workflow tests the actual code changes together

## Changes Made

- runtime-interface-client_merge_to_main.yml: Added local serialization build step
- runtime-interface-client_pr.yml: Added local serialization build step to both smoke-test and build jobs

This ensures CI/CD pipeline works correctly with the new dependency versions.
* Make handler response buffers thread safe.

* Add multiconcurrency tests

* ThreadLocal instead of Allocating new buffers every invoke.

* Thread local log4jContextPutMethod

* Fix indentations

* Add CountDownLatch to ensure all calls are done simultaneously.

---------

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
* Make Trace ID accessible through Context

* Change Constructor

---------

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
* RIC version bump to 2.8.6

---------

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
….8.7 (#110)

* Remove Miniumum and Maximum Limits of AWS_LAMBDA_MAX_CONCURRENCY

* RIC version Bump to 2.8.7

---------

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
#111)

Co-authored-by: Mohammed Ehab <moehabe@amazon.com>
@M-Elsaeed M-Elsaeed requested review from a user and maxday December 4, 2025 15:56
@darklight3it darklight3it self-requested a review December 4, 2025 16:16
@godcrampy godcrampy self-requested a review December 4, 2025 16:17
@M-Elsaeed M-Elsaeed removed request for a user, darklight3it and maxday December 4, 2025 16:29
@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

❌ Patch coverage is 64.78873% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.38%. Comparing base (5c6d3fa) to head (d1d42a2).
⚠️ Report is 27 commits behind head on main.

Files with missing lines Patch % Lines
.../lambda/runtime/api/client/EventHandlerLoader.java 45.00% 11 Missing ⚠️
.../client/runtimeapi/LambdaRuntimeApiClientImpl.java 60.86% 7 Missing and 2 partials ⚠️
...bda/runtime/api/client/util/ConcurrencyConfig.java 83.33% 2 Missing and 1 partial ⚠️
...a/runtime/api/client/logging/JsonLogFormatter.java 75.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #576      +/-   ##
============================================
+ Coverage     64.74%   65.38%   +0.63%     
- Complexity      194      211      +17     
============================================
  Files            33       34       +1     
  Lines           939      988      +49     
  Branches        132      142      +10     
============================================
+ Hits            608      646      +38     
- Misses          282      290       +8     
- Partials         49       52       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@M-Elsaeed M-Elsaeed merged commit 6b0be7c into main Dec 4, 2025
6 checks 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.

5 participants