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

Add foreign key constraints for all buildids #1657

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

williamjallen
Copy link
Collaborator

This PR builds upon the work in #1655, and adds foreign key constraints to all buildid columns. Like #1655, a handful of changes to our test suite were required to ensure that the constraints were fully met during the testing process.

This is related to issue #1442.

@williamjallen williamjallen marked this pull request as draft August 25, 2023 13:39
@williamjallen williamjallen force-pushed the build-foreign-keys branch 2 times, most recently from 68d1628 to e4bfb7e Compare August 28, 2023 22:00
@williamjallen williamjallen marked this pull request as ready for review August 29, 2023 14:03
@josephsnyder josephsnyder added this pull request to the merge queue Aug 30, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 30, 2023
@josephsnyder josephsnyder added this pull request to the merge queue Aug 30, 2023
Merged via the queue into Kitware:master with commit 8c059e3 Aug 30, 2023
2 checks passed
@williamjallen williamjallen deleted the build-foreign-keys branch August 30, 2023 14:19
zackgalbreath added a commit that referenced this pull request Mar 1, 2024
Follow-up to #1657

Remove DELETE statements that are now handled by cascading foreign
keys instead.
zackgalbreath added a commit that referenced this pull request Mar 1, 2024
Follow-up to #1657

Remove DELETE statements that are now handled by cascading foreign
keys instead.
github-merge-queue bot pushed a commit that referenced this pull request Mar 1, 2024
Follow-up to #1657

Remove DELETE statements that are now handled by cascading foreign keys
instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants