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

fix(migrations): CREATE EXTENSION deadlocks inside migrations global lock #210

Conversation

hsanjuan
Copy link
Contributor

@hsanjuan hsanjuan commented Nov 9, 2020

Instead, run this statement before obtaining the global lock for migrations.

Fixes #179 .

@hsanjuan
Copy link
Contributor Author

hsanjuan commented Nov 9, 2020

#179 says that this should documented to be run manually, but I find that suboptimal vs just doing it for the user (doing it when it's not needed does not seem like a bad thing either). If I'm missing something, let me know.

storage/migrate.go Outdated Show resolved Hide resolved
Run this statement directly, as it seems it cannot be run inside a
transaction.
@hsanjuan hsanjuan force-pushed the fix/179-remove-create-extension-from-migration branch from 544d0fa to 39f3611 Compare November 9, 2020 19:06
@hsanjuan hsanjuan self-assigned this Nov 9, 2020
@hsanjuan hsanjuan requested a review from iand November 9, 2020 19:07
@hsanjuan hsanjuan requested a review from iand November 17, 2020 19:39
@iand iand merged commit 1410a49 into filecoin-project:master Dec 4, 2020
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.

Remove create extension from migration
2 participants