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

CON-1185 backfill vacancies with ALE id #857

Merged
merged 1 commit into from
Dec 18, 2019

Conversation

DevenKShah
Copy link
Contributor

This is a data migration task. The code changes made here will be removed after the migration is done. Hence I have made precise changes only.
There are two new web jobs. Begin migration will trigger on queue, this will allow us to trigger migration when we are ready. The handler to this will queue up all the vacancies so that migration can take place in parallel.
The data migration listens to this queue and does the actual migration.
This setup will be reused for any future migrations as well.

@DevenKShah DevenKShah force-pushed the CON-1185-BackfillVacanciesWithALEId branch from 3ecc567 to 659f359 Compare December 13, 2019 17:13
Copy link
Contributor

@cofaulco cofaulco left a comment

Choose a reason for hiding this comment

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

There are a couple of style issues and small changes could be made, however as this is a one time migration, I don't want to get hung up on those

@DevenKShah DevenKShah force-pushed the CON-1185-BackfillVacanciesWithALEId branch from 659f359 to 6bf08d6 Compare December 17, 2019 09:16
@DevenKShah DevenKShah force-pushed the CON-1185-BackfillVacanciesWithALEId branch from 6bf08d6 to 32c312c Compare December 18, 2019 15:15
@HemanjaliCheruku HemanjaliCheruku merged commit 43b31f6 into master Dec 18, 2019
@HemanjaliCheruku HemanjaliCheruku deleted the CON-1185-BackfillVacanciesWithALEId branch December 18, 2019 15:21
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.

3 participants