Skip to content

fix: Foreign Keys in Snapshots#525

Merged
bbernays merged 3 commits into
cloudquery:mainfrom
bbernays:fix-fks-in-snapshot
Mar 9, 2022
Merged

fix: Foreign Keys in Snapshots#525
bbernays merged 3 commits into
cloudquery:mainfrom
bbernays:fix-fks-in-snapshot

Conversation

@bbernays
Copy link
Copy Markdown
Collaborator

@bbernays bbernays commented Mar 8, 2022

When restoring a snapshot prior to copying the data drop all FK requirements

@bbernays bbernays linked an issue Mar 8, 2022 that may be closed by this pull request
@bbernays bbernays changed the title Fix: Foreign Keys in Snapshots fix: Foreign Keys in Snapshots Mar 8, 2022
@github-actions github-actions Bot added the fix label Mar 8, 2022
Comment thread pkg/client/client.go Outdated
@bbernays bbernays requested a review from roneli March 8, 2022 20:02
Copy link
Copy Markdown
Contributor

@roneli roneli left a comment

Choose a reason for hiding this comment

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

Added a comment, other than that looks good.

Comment thread pkg/policy/snapshots.go
@bbernays bbernays requested review from disq and roneli March 9, 2022 15:53
@bbernays bbernays merged commit a4e289c into cloudquery:main Mar 9, 2022
@bbernays bbernays deleted the fix-fks-in-snapshot branch March 9, 2022 16:42
TinLe pushed a commit to TinLe/cloudquery that referenced this pull request Mar 22, 2022
* upstream/main: (21 commits)
  feat: Ignore ignored diags unless -v is specified (cloudquery#545)
  chore: Sync form .github (cloudquery#527)
  chore: Clean up db version checks (cloudquery#542)
  fix: Validate Timescale Version (cloudquery#540)
  feat: Support timeouts in fetch (cloudquery#537)
  fix: Warn on providers with no resources requested (cloudquery#528)
  fix: Fixed local policy run (cloudquery#535)
  fix: Make Date a static value (cloudquery#534)
  fix: Downgrade provider (cloudquery#531)
  feat: Store cq_fetch_id in meta (cloudquery#523)
  fix: Foreign Keys in Snapshots (cloudquery#525)
  feat: Add file() func to config parser (cloudquery#522)
  feat: Drift: Add @getbool modifier (cloudquery#515)
  fix: Diag: Don't report errors we don't care about (cloudquery#521)
  fix: Handle if Path doesn't exist (cloudquery#518)
  fix: Keep binary reference consistent (cloudquery#516)
  fix: Validate DB connection with an explicit timeout, rather than the… (cloudquery#510)
  fix: Missing views in history mode (cloudquery#508)
  fix: Init command cleanup (cloudquery#502)
  fix: Upgrade/downgrade providers cleanup (cloudquery#498)
  ...
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.

Can't Restore Tables with out FK data

3 participants