Skip to content

Example YAML configuration and Example DAG instantiation DAGs#75

Open
oscarrezab wants to merge 2 commits intomainfrom
ex_instantiations
Open

Example YAML configuration and Example DAG instantiation DAGs#75
oscarrezab wants to merge 2 commits intomainfrom
ex_instantiations

Conversation

@oscarrezab
Copy link

Add example configuration and instantiation for S3ToSnowflakeDag and SFTPToSnowflakeDag.

@oscarrezab oscarrezab requested a review from sleblanc23 June 17, 2025 21:28
@sleblanc23
Copy link
Contributor

Thanks for this Oscar! There are a few optional things I'd like to remove to make these a bit shorter. Could you please take out start_date and slack_conn_id from both YAML configurations, and also shorten the second config blocks to just two years/tenants for each DAG type? For example, remove 2020 and 2021 from the S3 DAG and lone_star and wiggins from the SFTP DAG.

@sleblanc23 sleblanc23 marked this pull request as ready for review June 20, 2025 18:17
Copy link
Contributor

@sleblanc23 sleblanc23 left a comment

Choose a reason for hiding this comment

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

Thanks, Oscar!

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.

2 participants