Skip to content

fix(migration): fix unexpected defer unlock#7634

Merged
klesh merged 1 commit into
mainfrom
dev0619-2
Jun 19, 2024
Merged

fix(migration): fix unexpected defer unlock#7634
klesh merged 1 commit into
mainfrom
dev0619-2

Conversation

@d4x1
Copy link
Copy Markdown
Contributor

@d4x1 d4x1 commented Jun 19, 2024

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

What does this PR do?
If err from err := migrator.Execute() is not nil, it will panic with fatal error: sync: unlock of unlocked mutex.
This PR just fix it, and panic with the correct error.

Does this close any open issues?

Closes N/A

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. pr-type/bug-fix This PR fixes a bug severity/p2 This bug doesn’t affect the functionality or isn’t evident labels Jun 19, 2024
@d4x1 d4x1 self-assigned this Jun 19, 2024
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jun 19, 2024
@klesh klesh merged commit 1e85c14 into main Jun 19, 2024
@klesh klesh deleted the dev0619-2 branch June 19, 2024 10:48
github-actions Bot pushed a commit that referenced this pull request Jun 19, 2024
@github-actions
Copy link
Copy Markdown
Contributor

🤖 Target: #release-v1.0 cherry pick finished successfully 🎉!

@github-actions github-actions Bot added the bot/auto-cherry-pick-completed auto cherry pick completed label Jun 19, 2024
d4x1 added a commit that referenced this pull request Jun 19, 2024
Co-authored-by: Lynwee <linwei.hou@merico.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot/auto-cherry-pick-completed auto cherry pick completed lgtm This PR has been approved by a maintainer needs-cherrypick-v1.0 pr-type/bug-fix This PR fixes a bug severity/p2 This bug doesn’t affect the functionality or isn’t evident size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants