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

Always backfill all GitHub data on every run #51

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Conversation

ghickman
Copy link
Contributor

While working on the GitHub data we realised that I had been backfilling after every major change to the code, and that the frequency at which we wrote the data was more than fast enough to do this in production too.

@ghickman ghickman force-pushed the always-be-backfillin branch 3 times, most recently from 9e3c6dd to 620667c Compare November 28, 2023 09:43
We're moving to replacing all the data every day, since that's what
we've ended up doing more often than not and it's already quite fast!
@ghickman ghickman force-pushed the always-be-backfillin branch from 620667c to 91c6289 Compare November 28, 2023 10:12
Copy link
Contributor

@benbc benbc left a comment

Choose a reason for hiding this comment

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

Doing a proper review later post-merge.

Base automatically changed from backfill-with-graphql to main November 28, 2023 12:25
@ghickman ghickman merged commit e5793a4 into main Nov 28, 2023
7 checks passed
@ghickman ghickman deleted the always-be-backfillin branch November 28, 2023 12:25
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