Skip to content

Try fix crash when server startup with unavailable MySQL server#16132

Closed
zhang2014 wants to merge 26 commits intoClickHouse:masterfrom
zhang2014:fix/crash_when_mysql_error
Closed

Try fix crash when server startup with unavailable MySQL server#16132
zhang2014 wants to merge 26 commits intoClickHouse:masterfrom
zhang2014:fix/crash_when_mysql_error

Conversation

@zhang2014
Copy link
Copy Markdown
Contributor

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Try fix crash when server startup with unavailable MySQL server & support drop MySQL database engine with unavailable MySQL server.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Oct 19, 2020
@tavplubix tavplubix self-assigned this Oct 19, 2020
@zhang2014
Copy link
Copy Markdown
Contributor Author

Integration tests (asan) — Some exception occurred :(
Integration tests (thread) — Some exception occurred :(

Unknown reason.

@tavplubix
Copy link
Copy Markdown
Member

Try fix crash when server startup with unavailable MySQL server & support drop MySQL database engine with unavailable MySQL server.

Could you please also add an integration test for this?

@zhang2014
Copy link
Copy Markdown
Contributor Author

Could you please also add an integration test for this?

Ok, I will add it .

@zhang2014 zhang2014 force-pushed the fix/crash_when_mysql_error branch from 701b20e to a748dad Compare November 10, 2020 09:52
@zhang2014 zhang2014 force-pushed the fix/crash_when_mysql_error branch from 533790e to fb80231 Compare November 12, 2020 03:47
@qoega
Copy link
Copy Markdown
Member

qoega commented Feb 1, 2021

@zhang2014 will you have time to return to this PR?

@alexey-milovidov
Copy link
Copy Markdown
Member

Looks like this PR is abandoned.

@beeflyme
Copy link
Copy Markdown

beeflyme commented Oct 8, 2021

any update abt this problem?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants