-
Notifications
You must be signed in to change notification settings - Fork 1k
first commit - lambda-durable-functions-nodejs-sam #2935
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
base: main
Are you sure you want to change the base?
first commit - lambda-durable-functions-nodejs-sam #2935
Conversation
| { | ||
| "name": "nodejs-durable-order-processor", | ||
| "version": "1.0.0", | ||
| "description": "AWS Lambda Durable Functions - Node.js Orchestrator with structured logging and error handling", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "description": "AWS Lambda Durable Functions - Node.js Orchestrator with structured logging and error handling", | |
| "description": "AWS Lambda durable functions - Node.js orchestrator with structured logging and error handling", |
| @@ -0,0 +1,63 @@ | |||
| { | |||
| "title": "AWS Lambda Durable Functions with Node.js", | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "title": "AWS Lambda Durable Functions with Node.js", | |
| "title": "AWS Lambda durable functions with Node.js", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make sure to always use the correct capitalization
| @@ -0,0 +1,63 @@ | |||
| { | |||
| "title": "AWS Lambda Durable Functions with Node.js", | |||
| "description": "Demonstrates AWS Lambda Durable Functions using Node.js with automatic checkpointing, durable waits, and fault tolerance for long-running workflows.", | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "description": "Demonstrates AWS Lambda Durable Functions using Node.js with automatic checkpointing, durable waits, and fault tolerance for long-running workflows.", | |
| "description": "Demonstrates AWS Lambda durable functions using Node.js with automatic checkpointing, durable waits, and fault tolerance for long-running workflows.", |
| }, | ||
| "cleanup": { | ||
| "text": [ | ||
| "Delete the stack: sam delete --stack-name durable-functions-demo" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "Delete the stack: sam delete --stack-name durable-functions-demo" | |
| "Delete the stack: <code>sam delete --stack-name durable-functions-demo</code>" |
| @@ -0,0 +1,279 @@ | |||
| # AWS Lambda Durable Functions with Node.js | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # AWS Lambda Durable Functions with Node.js | |
| # AWS Lambda durable functions with Node.js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make sure to always use the correct capitalization
| @@ -0,0 +1,112 @@ | |||
| AWSTemplateFormatVersion: '2010-09-09' | |||
| Transform: AWS::Serverless-2016-10-31 | |||
| Description: AWS Lambda Durable Functions - Node.js Order Processing Demo | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check correct name
|
|
||
| Globals: | ||
| Function: | ||
| Runtime: nodejs22.x |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not the latest runtime?
| Globals: | ||
| Function: | ||
| Runtime: nodejs22.x | ||
| Timeout: 900 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is such a long timeout necessary?
| Timeout: 900 | ||
| MemorySize: 512 | ||
| Architectures: | ||
| - x86_64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not arm64?
|
|
||
| Resources: | ||
| # IAM Role for Lambda Functions | ||
| LambdaDurableExecutionRole: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These permissions are overly broad. Scope them to the relevant resources
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.