Skip to content
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

chore(pipeline) : Make the main DAG run hourly #300

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Conversation

vperron
Copy link
Contributor

@vperron vperron commented Sep 19, 2024

Having two DAGS running the geocodages model in the same time doesn't bode well with the incremental models.

Let's switch to running the main DAG hourly, which also kind of makes sense as our pipeline is more frequently updated and is also by far the simpler option, thus the one with the fewer surprises.

Having two DAGS running the geocodages model in the same time doesn't
bode well with the incremental models.

Let's switch to running the main DAG hourly, which also kind of makes
sense as our pipeline is more frequently updated and is also by far the
simpler option, thus the one with the fewer surprises.
@vperron vperron requested a review from vmttn as a code owner September 19, 2024 08:03
@vperron vperron requested review from hlecuyer and vmttn and removed request for vmttn September 19, 2024 08:03
@vperron vperron self-assigned this Sep 19, 2024
Copy link
Contributor

@vmttn vmttn left a comment

Choose a reason for hiding this comment

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

il faudrait retirer les "schedule": "@..." du fichier dags/dag_utils/sources/__init__.py

@vperron
Copy link
Contributor Author

vperron commented Sep 19, 2024

Négatif, ils sont utilisés par import_sources.py

schedule=source_config["schedule"],

@vmttn
Copy link
Contributor

vmttn commented Sep 19, 2024

roger that

@vperron vperron merged commit 6a9d8d7 into main Sep 19, 2024
7 of 9 checks passed
@vperron vperron deleted the vperron/main-hourly branch September 19, 2024 09:51
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