-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Fix segfault on attach parititon #31738
Conversation
ad7fe45
to
1ab8b5d
Compare
{ | ||
String old_name; | ||
String new_name; | ||
DiskPtr disk; |
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.
As I can guess, both old and new part should be on the same disk?
Maybe add a comment here ...
export -f thread_detach_attach; | ||
export -f thread_detach_attach; |
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.
thread_drop_detached?
timeout $TIMEOUT bash -c thread_detach_attach 2> /dev/null & | ||
timeout $TIMEOUT bash -c thread_detach_attach 2> /dev/null & | ||
timeout $TIMEOUT bash -c thread_detach_attach 2> /dev/null & |
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.
thread_drop_detached somewhere?
9d94b5a
to
2cf4b51
Compare
Backport #31738 to 21.11: Fix segfault on attach parititon
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixed rare segfault on concurrent
ATTACH PARTITION
queries.Detailed description / Documentation draft:
Also removed some bad code
https://clickhouse-test-reports.s3.yandex.net/30599/200913a5a575ffb03a0307c68da4162c9844acbb/stress_test_(address).html#fail1