-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 zone lost in backup #4319
fix zone lost in backup #4319
Conversation
if (zoneRet != nebula::cpp2::ErrorCode::E_ZONE_NOT_FOUND) { | ||
if (zoneRet == nebula::cpp2::ErrorCode::SUCCEEDED) | ||
LOG(ERROR) << "Zone " << zoneName << " have existed"; | ||
code = nebula::cpp2::ErrorCode::E_KEY_HAS_EXISTS; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps you need to return the error code from zoneRet, otherwise, some kvstore error could be vanished. Besides, the if
in line 64 miss a bracket
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thx.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job
5b81dee
to
4e86a46
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job~
82264f9
to
c1d402f
Compare
What type of PR is this?
What problem(s) does this PR solve?
Issue(s) number:
Description:
Solve the problem of zonename inconsistency during restore.
Currently a host is a zone by default.
When restoring, because the host needs to be replaced, so the zone needs to be replaced.
And when replacing the zone, need to replace the zone used in spaceDesc.
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc:
Checklist:
Tests:
Affects:
Release notes:
Please confirm whether to be reflected in release notes and how to describe: