-
Notifications
You must be signed in to change notification settings - Fork 487
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
Schema dump, PROCESS privilege #250
Comments
I think we should include tablespaces. I'll check to see if we can add that privilege to Sail. |
Btw I see you're running |
@driesvints I did actually use |
@ethanclevenger91 are you able to make a PR to fix this problem? |
@taylorotwell probably, with a firm pick on the direction you want to go (ignore tablespaces or add permissions) |
Probably add permissions as I don't think that skipping part of the schema is a good choice? It would effectively make the database different from the original. |
Knowing very little about tablespaces, I'm not convinced they're important in this context, but I'm not an expert in that space. Is it even possible to manipulate tablespaces via Laravel migrations? |
@ethanclevenger91 yeah I think you may be right after all. Maybe we should consider ignoring tablespaces. If you can, feel free to send that to the framework. |
I'll have a go. The other thing that occurred to me - by doing this, MySQL is going to break a lot of people's backup systems, right? The framework can't make assumptions about what users have what permissions, so simply running a command most likely to succeed can mitigate some of that. |
Oh, and maybe move this issue back to the framework, for posterity? |
Keeping this open here as I'm also running into this now. |
Description:
php artisan schema:dump --prune
throws error:Appears to have run successfully, but it wouldn't be the first time I got a partial backup due to a MySQL error during an operation.
The two possible solutions seem to be:
mysqldump
with the--no-tablespaces
option. Simpler option, especially if tablespaces aren't vital to this operation.Steps To Reproduce:
php artisan schema:dump --prune
to turn your migration into a schema file.The text was updated successfully, but these errors were encountered: