Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

fix disk manager thread conflict #512

Merged
merged 5 commits into from
Jul 6, 2021
Merged

Conversation

critical27
Copy link
Contributor

@critical27 critical27 commented Jul 5, 2021

Fix thread conflict when multiple thread call addPartToPath.

See ASAN report in #508

@critical27 critical27 added bug Type: something is unexpected ready-for-testing PR: ready for the CI test labels Jul 5, 2021
panda-sheep
panda-sheep previously approved these changes Jul 5, 2021
Copy link
Contributor

@bright-starry-sky bright-starry-sky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done.

@critical27
Copy link
Contributor Author

Hold on a second, modify a parameter default value.

panda-sheep
panda-sheep previously approved these changes Jul 6, 2021
Copy link
Contributor

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good!

@laura-ding
Copy link
Contributor

close #514

@critical27
Copy link
Contributor Author

Hold on one second again.....

@critical27 critical27 merged commit 3842980 into vesoft-inc:master Jul 6, 2021
@critical27 critical27 deleted the fix branch July 6, 2021 06:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Type: something is unexpected ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants