New serverless pattern - apigw-python-cdk-lambda-snapstart#2957
New serverless pattern - apigw-python-cdk-lambda-snapstart#2957mate329 wants to merge 7 commits intoaws-samples:mainfrom
Conversation
Co-authored-by: ellisms <114107920+ellisms@users.noreply.github.com>
Co-authored-by: ellisms <114107920+ellisms@users.noreply.github.com>
Co-authored-by: ellisms <114107920+ellisms@users.noreply.github.com>
Co-authored-by: ellisms <114107920+ellisms@users.noreply.github.com>
… + clearer instructions for deployment
| pip3 install -r requirements.txt | ||
|
|
||
| # Install AWS Lambda Powertools library for the CarHandler Lambda | ||
| pip3 install -r CarHandler/requirements.txt -t CarHandler/ |
There was a problem hiding this comment.
hello @ellisms thank you very much for reviewing my PR, sir!
one question for you please - i've put the PIP command here in instructions to install aws-lambda-powertools as you've suggested. what do you think is better - keeping the installation procedure of the library like this, or creating a Lambda layer inside the CDK code?
thank you in advance!
There was a problem hiding this comment.
I think either is ok. In either case, I would lock in the version number so that the user is confident they will get a known-working configuration.
There was a problem hiding this comment.
understood, then i'd like to keep it as is right now. the version number is locked, so this part of the PR should be good to go. thanks!
Issue #, if available:
Description of changes:
This project is a serverless car store REST API built with AWS CDK (Python) that uses API Gateway, Lambda with SnapStart for improved cold start performance, and DynamoDB to provide CRUD operations on car records (make, model, year, color) through /cars endpoints, with the Lambda handler managing HTTP routing, JSON parsing, and DynamoDB operations for create, read, update, and delete functionality.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.