-
-
Notifications
You must be signed in to change notification settings - Fork 501
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
Pomelo Entity Framework connector is not able to commit changes #7897
Comments
Hi @awarycha, thanks for opening a new issue for this one. This looks related to a recent change @jycor made to show database level changes (e.g. a collation change) as changed tables. Here's an even simpler repro from a show create database may24;
+----------+--------------------------------------------------------------------------------------------+
| Database | Create Database |
+----------+--------------------------------------------------------------------------------------------+
| may24 | CREATE DATABASE `may24` /*!40100 DEFAULT CHARACTER SET utf8mb4 COLLATE utf8mb4_0900_bin */ |
+----------+--------------------------------------------------------------------------------------------+
alter database may24 collate utf8mb4_0900_ai_ci;
Query OK, 1 row affected (0.00 sec)
select * from dolt_status;
+-------------------+--------+----------+
| table_name | staged | status |
+-------------------+--------+----------+
| __DATABASE__may24 | false | modified |
+-------------------+--------+----------+
call dolt_commit('-a', '-m', 'changing collation', '--author', 'system <system@domain.com>');
error: the table(s) __DATABASE__may24 do not exist I'll follow up with @jycor and see if he can pick this one up today. It should be a quick fix. |
Hey @awarycha, Just merged a fix for this to dolt main. I had missed this particular case when adding this change. |
Hi @awarycha, we have released Dolt version 1.39.1 with the fix for this issue. Sorry for the snag on this regression, and thank you for taking the time to report it to us so we could fix it. 🙏 I hope testing your ASP.NET Core app with this new version goes smoothly! Let us know if you hit any more issues and we'll be happy to help. |
Thanks for the quick fix. I am not able to test it now but I will do it on monday morning and then give you a feedback. |
Verified on Dolt version 1.39.1. Works as expected. Thanks. |
Problem exist on Dolt in Docker both dolthub/dolt-sql-server:1.38.3 and dolthub/dolt-sql-server:1.39.0 only with Pomelo connector
Pomelo.EntityFrameworkCore.MySql" Version="8.0.2", everything works if I use MySql.EntityFrameworkCore" Version="8.0.2" connector.
I tested also commit on Pomelo and Dolt dolthub/dolt-sql-server:1.34.0 and it is working fine. So I think it is regress, please confirm.
Steps to reproduce:
At the beginning of application after recreation of database scheme, I'm adding initial commit
Then I get the following error:
The text was updated successfully, but these errors were encountered: