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

Improve error messaging when multiple processes connect to db concurrently. #5793

Closed
nicktobey opened this issue Apr 24, 2023 · 1 comment
Closed
Assignees
Labels
bad error message sql server Issues related to the built in SQL server

Comments

@nicktobey
Copy link
Contributor

Today I tried to run some CLI commands on a database while I had a debug process suspended in my IDE.

The error message read:

error on line 1 for query DROP TABLE t: cannot update manifest: database is read only
cannot update manifest: database is read only

It's good that we detect and disallow concurrent modifications to the database, but our messaging could be better. Is it possible for the CLI to distinguish between a db that's locked because of concurrent access and a db that's actually read-only?

@timsehn timsehn added bad error message sql server Issues related to the built in SQL server labels Apr 25, 2023
@timsehn
Copy link
Contributor

timsehn commented Apr 27, 2023

@macneale4 will fix this with his current work as you'll be able to connect with the CLI to a running database.

Gonna close it in favor of the uber issue #3922

@timsehn timsehn closed this as not planned Won't fix, can't repro, duplicate, stale Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bad error message sql server Issues related to the built in SQL server
Projects
None yet
Development

No branches or pull requests

3 participants