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

Creation of a database with a name identical to a recently dropped-and-dolt_purge_dropped_database() database fails on Windows #7492

Closed
gitasmus opened this issue Feb 14, 2024 · 4 comments · Fixed by #7494
Labels
bug Something isn't working customer issue windows For Windows specific bugs

Comments

@gitasmus
Copy link

On Windows, the newest dolt release https://github.com/dolthub/dolt/releases/download/v1.34.0/dolt-windows-amd64.msi causes the problem described in the subject line of this issue.

create database to_be_purged;
drop database to_be_purged;
call dolt_purge_dropped_databases();
create database to_be_purged;

returns the error

error reading from or writing to noms: cannot update manifest: database is read only

On Linux, there is no such problem for 1.34.0.

@gitasmus
Copy link
Author

Addition: On Windows you get this error regardless of whether you have called dolt_purge_dropped_databases() before the create database command.

@fulghum fulghum added bug Something isn't working windows For Windows specific bugs labels Feb 14, 2024
@fulghum
Copy link
Contributor

fulghum commented Feb 14, 2024

Hi @gitasmus, thank you for taking the time to report this one to us. 🙏 We'll get someone on our team with a Windows laptop to dig into this one today and see what's going on.

@fulghum
Copy link
Contributor

fulghum commented Feb 14, 2024

Okay, @jycor is going to try and repro this one today on Windows. We'll let you know what we find.

@jycor
Copy link
Contributor

jycor commented Feb 14, 2024

Hey @gitasmus,

Thanks for reporting this issue! It turns out we weren't clearing some cache correctly.
I was able to reproduce and fix the problem; a PR with the fix is currently out for review.
We'll try to get a release out ASAP, likely later today or tomorrow.

A workaround for now is to exit the shell right after dropping the database.

$ dolt sql
# Welcome to the DoltSQL shell.
# Statements must be terminated with ';'.
# "exit" or "quit" (or Ctrl-D) to exit.
tmp/main> create database test_db;
tmp/main> drop database test_db;
tmp/main> exit
Bye

$ dolt sql
# Welcome to the DoltSQL shell.
# Statements must be terminated with ';'.
# "exit" or "quit" (or Ctrl-D) to exit.
tmp/main> create database test_db;
tmp/main> exit
Bye

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working customer issue windows For Windows specific bugs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants