-
Notifications
You must be signed in to change notification settings - Fork 2.6k
perf(etcd): refactor some code to improve performance #12011
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: xuruidong <xuruidong@gmail.com>
@xuruidong Just approved to run the CI |
please run the CI again @juzhiyuan |
Hi @xuruidong, I couldn't find the Re-run button. Can you confirm the failed test cases? ![]() |
I don't think the CI failure is related to the code changes. |
Hi @xuruidong, can you add some descriptive information for PR? |
Hi @xuruidong, please add a PR description so that others can review it better ~ |
Hi @Baoyuantop , the PR description has been added, please help review it. Thanks. |
Please fix the failed ci |
Hi @Baoyuantop ,please trigger the ci |
Hi @Baoyuantop , the ci is not stable, the failed test should be rerun |
Description
Fixes # (issue)
Before:
The original code performed a cleanup of
self.values
and rebuiltself.values_hash
by:self.values
nil
valuesvalues_hash
in a separate stepProblems:
Now:
nil
values while keeping the same tablevalues_hash
at the same timeWhy better?
Before:
The original code performed a cleanup of
self.values
and rebuiltself.values_hash
by:self.values
nil
valuesvalues_hash
in a separate stepProblems:
Now:
nil
values while keeping the same tablevalues_hash
at the same timeWhy better?
Checklist