You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/06/06 15:10:21 UTC

[GitHub] [airflow] vincbeck commented on pull request #24240: AIP-47 - Migrate s3 DAGs to new design #22438

vincbeck commented on PR #24240:
URL: https://github.com/apache/airflow/pull/24240#issuecomment-1147556792

   Whao! Thanks for doing that @chethanuk-plutoflume . One general comment would be, we would like each sample dag to be self contained, meaning each sample dag creates and deletes all the resources/infrastructure it is using. One example would be in `example_dynamodb_to_s3.py` to create the S3 bucket and Dynamo table as setup and remove them as teardown. A good example is #24058 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org