-
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
Support zone operations #3444
Support zone operations #3444
Conversation
8e2305d
to
1c087b5
Compare
c040cf8
to
d9e663e
Compare
Codecov Report
@@ Coverage Diff @@
## master #3444 +/- ##
==========================================
- Coverage 85.19% 79.14% -6.06%
==========================================
Files 1306 1251 -55
Lines 122158 105097 -17061
==========================================
- Hits 104078 83174 -20904
- Misses 18080 21923 +3843
Continue to review full report at Codecov.
|
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.
Please check the lock in all processors
} | ||
folly::Baton<true, std::atomic> baton; | ||
auto result = nebula::cpp2::ErrorCode::SUCCEEDED; | ||
kvstore_->asyncMultiRemove(kDefaultSpaceId, |
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.
Why don't use LogBatch? You can't promise it is atomic. So as DropHost
and other processor, please check it
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.
Check all processor zone related, e.g. DropHostProcessor
in previous PR
b6b8f8d
to
fb78eb1
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.
Well done~
3267003
to
9880cf3
Compare
Don't merge pls |
bdf2fa7
bdf2fa7
to
0d5cda7
Compare
What type of PR is this?
What does this PR do?
Which issue(s)/PR(s) this PR relates to?
Special notes for your reviewer, ex. impact of this fix, etc:
Additional context:
Checklist:
Release notes:
Please confirm whether to reflect in release notes and how to describe: