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

Removed columns should be made null-able #104

Open
mitar opened this issue Apr 28, 2021 · 1 comment
Open

Removed columns should be made null-able #104

mitar opened this issue Apr 28, 2021 · 1 comment

Comments

@mitar
Copy link

mitar commented Apr 28, 2021

So I think it is OK that removed columns are kept. But they should be changed so that NULL values can be inserted into them. This means that future queries which do not know about those columns can succeed and database will just insert NULL values in their place.

@mitar
Copy link
Author

mitar commented Apr 28, 2021

It would also be useful to rename them, something like _<old name>_<deletion timestamp> so that any query assuming they are still around would start failing (one can detect bugs this way) and that any future column which happens to share the name (when you forget that there was already a column with this name) does not conflict.

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

No branches or pull requests

1 participant