-
Notifications
You must be signed in to change notification settings - Fork 410
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
Storages: fix a series of data race or local index related issues #9962
base: master
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
dbms/src/Storages/DeltaMerge/DeltaMergeStore_InternalSegment.cpp
Outdated
Show resolved
Hide resolved
982b18e
to
8120f7c
Compare
Signed-off-by: Lloyd-Pottiger <[email protected]>
Signed-off-by: Lloyd-Pottiger <[email protected]>
18d1047
to
5c5d832
Compare
dbms/src/Storages/DeltaMerge/DeltaMergeStore_InternalSegment.cpp
Outdated
Show resolved
Hide resolved
Signed-off-by: Lloyd-Pottiger <[email protected]>
I think you need a separate issue and fix PR for this problem |
Signed-off-by: Lloyd-Pottiger <[email protected]>
Open a separate issue, but IMO fix in this PR is ok, since other changes also need pick to release-8.5. |
Signed-off-by: Lloyd-Pottiger <[email protected]>
What problem does this PR solve?
Issue Number: close #9961, close #9971, close #9972
Problem Summary:
What is changed and how it works?
Check List
Tests
Side effects
Documentation
Release note