Skip to content

Better handling of the table initialization #940

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

Merged
merged 2 commits into from
Mar 6, 2020
Merged

Conversation

goetas
Copy link
Member

@goetas goetas commented Mar 6, 2020

Q A
Type improvement
BC Break -
Fixed issues #935

Summary

This PR solves #935.

The idea behind is that:

  • if there is no migration table then we consider it as if there are no executed migrations (in this case all the read command work as usual, the write command will create the table as expected on the very first migrations run)
  • if there is a table but is not up to date, any command will trigger an exception and the user will have to run the sync-metadata command

With this PR the read commands will never update your DB and the write commands will do so only after the user's consent

@goetas goetas added this to the 3.0.0 milestone Mar 6, 2020
@goetas goetas force-pushed the no-schema-table branch from 92f1cd5 to 3d578fd Compare March 6, 2020 21:35
@greg0ire greg0ire changed the title Better handling of the table intialization Better handling of the table initialization Mar 6, 2020
@goetas goetas force-pushed the no-schema-table branch from 3d578fd to 3e6768c Compare March 6, 2020 21:41
@goetas goetas merged commit 05628ad into master Mar 6, 2020
@goetas goetas deleted the no-schema-table branch March 6, 2020 21:53
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