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

handle foreign key violation when expected record doesnt exist #9016

Merged
merged 4 commits into from
Dec 21, 2021

Conversation

subodh1810
Copy link
Contributor

Ref : https://airbytehq.slack.com/archives/C0229LM09T4/p1640117342274300

The migration fails cause of foreign key violation.

@subodh1810 subodh1810 self-assigned this Dec 21, 2021
@subodh1810 subodh1810 temporarily deployed to more-secrets December 21, 2021 21:31 Inactive
@subodh1810 subodh1810 temporarily deployed to more-secrets December 21, 2021 21:45 Inactive
@subodh1810 subodh1810 temporarily deployed to more-secrets December 21, 2021 21:53 Inactive
@subodh1810 subodh1810 temporarily deployed to more-secrets December 21, 2021 22:10 Inactive
Copy link
Contributor

@cgardens cgardens left a comment

Choose a reason for hiding this comment

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

is there any way to do a test run of this script?

@subodh1810
Copy link
Contributor Author

subodh1810 commented Dec 21, 2021

@cgardens I have tested this for OSS by creating a fresh instance of airbyte and also migrating from a pre-existing instance of airbyte to a newer version. For cloud I was thinking of testing this by deploying it on dev.

@cgardens
Copy link
Contributor

okay! sounds good.

@subodh1810 subodh1810 merged commit a6923b2 into master Dec 21, 2021
@subodh1810 subodh1810 deleted the handle-foregin-key-violation branch December 21, 2021 23: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.

None yet

2 participants