Skip to content

Conversation

@architec
Copy link
Contributor

Shows how to use Airflow with Lambda, and DynamoDB for human approval.

Airflow calls Lambda
Airflow adds entry to DDB, waiting for value change from DDB to continue in workflow


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

@architec architec changed the title feat:(typescript/airflow-lambda-dynamodb-approval) feat(typescript):airflow-lambda-dynamodb-approval Sep 3, 2025
Copy link
Contributor

@kaiz-io kaiz-io left a comment

Choose a reason for hiding this comment

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

REQUEST CHANGES: Missing critical infrastructure file

The main CDK stack file (lib/aws-mwaa-cdk-stack.ts) is missing from this PR.

import { AwsMwaaCdkStack } from '../lib/aws-mwaa-cdk-stack';

const app = new cdk.App();
new AwsMwaaCdkStack(app, 'AwsMwaaCdkStack');
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kaiz-io Fixed. Added lib/aws-mwaa-cdk-stack.ts file in commit 267dabc.

@kaiz-io kaiz-io added the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. label Oct 9, 2025
kaiz-io and others added 3 commits October 9, 2025 14:37
…odb-approval

- Add lib/aws-mwaa-cdk-stack.ts with complete MWAA infrastructure
- Update .gitignore to track TypeScript source files in lib/
- Stack includes VPC, S3, Lambda, DynamoDB, and MWAA environment
- Addresses PR aws-samples#1207 review feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants