Uh oh!
There was an error while loading. Please reload this page.
[refactor](schema change) Using tablet schema shared ptr instead of raw ptr - #11475
Conversation
Have you done a stress test to see whether this is a performance drop for this pr? |
yiguolei
commented
Aug 4, 2022
No, because tablet schema is only used in control path, it is not on data path. And there will be a nightly performance benchmark in selectdb. If we find performance drops we could revert this PR. |
Lchangliang
commented
Aug 4, 2022
LGTM |
yiguolei
commented
Aug 4, 2022
@wangbo I have done a simple performance test.
compare current branch with a47eff1 commit. |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Proposed changes
This is part of light weight schema change #10136
Problem summary
Describe your changes.
Checklist(Required)
Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...