Skip to content

[IOTDB-1619] There is an error msg when I restart iotdb-cluster#3930

Merged
neuyilan merged 1 commit intoapache:masterfrom
cigarl:raft
Sep 9, 2021
Merged

[IOTDB-1619] There is an error msg when I restart iotdb-cluster#3930
neuyilan merged 1 commit intoapache:masterfrom
cigarl:raft

Conversation

@cigarl
Copy link
Copy Markdown
Contributor

@cigarl cigarl commented Sep 8, 2021

When i try to figure out why this issue occurs, i find that the result of last raft log file is different with my guess.Here are the result of my test environment (ps.I ran into the same problem.)

We find these files of raft log:

/.../system/raftLog/-1869016501/49923372-9223372036854775807-244-idx, 
/.../system/raftLog/-1869016501/48875442-49224751-241-idx, 
/.../system/raftLog/-1869016501/49224752-49574061-242-idx, 
/.../system/raftLog/-1869016501/49574062-49923371-243-idx

/.../system/raftLog/-1869016501/49574062-49923371-243-data, 
/.../system/raftLog/-1869016501/49923372-9223372036854775807-244-data, 
/.../system/raftLog/-1869016501/48875442-49224751-241-data, 
/.../system/raftLog/-1869016501/49224752-49574061-242-data

and process these files (242-index and 243-data?) :
image

Obviously, we have a problem with where the files are sorted.That may be a bomb in recover module.

footnote Here's meta info

commitLogTerm=1, 
commitLogIndex=49923370, 
lastLogIndex=49923370, 
lastLogTerm=1, 
maxHaveAppliedCommitIndex=49923357

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.005%) to 67.417% when pulling db9a73f on cigarl:raft into aaa96b9 on apache:master.

@OneSizeFitsQuorum OneSizeFitsQuorum added the Module - Cluster PRs for the cluster module label Sep 8, 2021
Copy link
Copy Markdown
Member

@neuyilan neuyilan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@neuyilan
Copy link
Copy Markdown
Member

neuyilan commented Sep 9, 2021

cherry-pick to rel/0.12 please

@neuyilan neuyilan merged commit 2f279dd into apache:master Sep 9, 2021
cigarl added a commit to cigarl/iotdb that referenced this pull request Sep 9, 2021
@cigarl
Copy link
Copy Markdown
Contributor Author

cigarl commented Sep 9, 2021

cherry-pick to rel/0.12 please

Done, see #3933

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Module - Cluster PRs for the cluster module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants