Skip to content

Avoid error after upgrading a database#1519

Merged
aeisenberg merged 2 commits intomainfrom
aeisenberg/upgrade-db-fix
Sep 20, 2022
Merged

Avoid error after upgrading a database#1519
aeisenberg merged 2 commits intomainfrom
aeisenberg/upgrade-db-fix

Conversation

@aeisenberg
Copy link
Copy Markdown
Contributor

@aeisenberg aeisenberg commented Sep 16, 2022

The runUpgrade query server command is mistakenly caching the old dbscheme in memory after running the upgrade. The problem is in the CLI. The workaround is to restart the query server after running an upgrade. This is not a great solution, but considering that explicit upgrades are now very rare. I do not think it is worth putting in too much effort for a proper fix.

Replace this with a description of the changes your pull request makes.

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

The `runUpgrade` query server command is mistakenly caching the old
dbscheme in memory after running the upgrade. The problem is in the
CLI. The workaround is to restart the query server after running an
upgrade. This is not a great solution, but considering that explicit
upgrades are now very rare. I do not think it is worth putting in
too much effort for a proper fix.
@aeisenberg aeisenberg requested a review from a team as a code owner September 16, 2022 23:16
@aeisenberg
Copy link
Copy Markdown
Contributor Author

@alexet I am merging this now, but if you want to look at the underlying issue later, we can put in a proper fix at some point.

@aeisenberg aeisenberg merged commit 60c4d8d into main Sep 20, 2022
@aeisenberg aeisenberg deleted the aeisenberg/upgrade-db-fix branch September 20, 2022 16:38
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.

2 participants