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: fix a crash bug when open an incomplete created RocksDB instance #1451

Merged
merged 3 commits into from
Apr 21, 2023

Conversation

acelyc111
Copy link
Member

@acelyc111 acelyc111 commented Apr 17, 2023

#1450

If replica server attempt to open an incomplete RocksDB instance (maybe caused
by a previous crash), it will crash before moving the incomplete path to ".err"
trash path, and it will crash again if restart the server.

This patch avoid to crash before moving the incomplete RocksDB path to ".err" path,
thus the replica has an opportunity to recovery automatically without move the
incomplete RocksDB path manually.

@acelyc111 acelyc111 marked this pull request as ready for review April 21, 2023 02:52
@acelyc111 acelyc111 changed the title fix: Avoid to crash while load an imcomplete RocksDB instance fix: Avoid to crash while load an incomplete RocksDB instance Apr 21, 2023
@acelyc111 acelyc111 changed the title fix: Avoid to crash while load an incomplete RocksDB instance fix: fix a crash bug when open an incomplete RocksDB instance Apr 21, 2023
@acelyc111 acelyc111 changed the title fix: fix a crash bug when open an incomplete RocksDB instance fix: fix a crash bug when open an incomplete created RocksDB instance Apr 21, 2023
@empiredan empiredan merged commit 7a6c28e into apache:master Apr 21, 2023
@empiredan empiredan mentioned this pull request Aug 21, 2023
GehaFearless pushed a commit to GehaFearless/incubator-pegasus that referenced this pull request Feb 28, 2024
…B instance (apache#1451)

对应社区commit: https://github.com/apache/incubator-pegasus/pull/1451/files

注: 由于单测部分变更较大,本次未合入

apache#1450

If replica server attempt to open an incomplete RocksDB instance (maybe caused
by a previous crash), it will crash before moving the incomplete path to ".err"
trash path, and it will crash again if restart the server.

This patch avoid to crash before moving the incomplete RocksDB path to ".err" path,
thus the replica has an opportunity to recovery automatically without move the
incomplete RocksDB path manually.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants