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 a bug may stuck #3306

Merged
merged 3 commits into from
Nov 12, 2021
Merged

fix a bug may stuck #3306

merged 3 commits into from
Nov 12, 2021

Conversation

liuyu85cn
Copy link
Contributor

What type of PR is this?

  • bug
  • feature
  • enhancement

What does this PR do?

fix an issue that storage may stuck.

Which issue(s)/PR(s) this PR relates to?

#3304

Special notes for your reviewer, ex. impact of this fix, etc:

Additional context:

we need 3 pre-condition to repro this:

1: storage server is stopping
2. a resume processor is running.
3. the previous processor throw an exception.

Checklist:

  • Documentation affected (Please add the label if documentation needs to be modified.)
  • Incompatible (If it is incompatible, please describe it and add corresponding label.)
  • Need to cherry-pick (If need to cherry-pick to some branches, please label the destination version(s).)
  • Performance impacted: Consumes more CPU/Memory

Release notes:

Please confirm whether to reflect in release notes and how to describe:

                                                            `

@liuyu85cn liuyu85cn added the ready-for-testing PR: ready for the CI test label Nov 11, 2021
@yixinglu yixinglu linked an issue Nov 12, 2021 that may be closed by this pull request
@yixinglu yixinglu merged commit 2c1d720 into vesoft-inc:master Nov 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TOSS add edge block storaged from exit
3 participants