Skip to content
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

reload @@config_all twice error #217

Closed
actiontech-bot opened this issue Sep 3, 2017 · 3 comments
Closed

reload @@config_all twice error #217

actiontech-bot opened this issue Sep 3, 2017 · 3 comments

Comments

@actiontech-bot
Copy link
Member

actiontech-bot commented Sep 3, 2017

Raised by: @zhunina
The second reload @@config_all will return error: ERROR 1003 (HY000):The before reload @@config_all has an unfinished db transaction,please try again later

@actiontech-bot
Copy link
Member Author

By @yanhuqing666

Before:
reload @@config_all will recycle all connections async.
It will try to recycle only the timeout (sqlExecuteTimeout, default is 300s ) connections every 5s.
Before all connections are recycled, the second reload @@config_all will throw error.

Fixed:
reload @@config_all will recycle processing connections async.
It will try to recycle connections which is timeout or whose processing transaction is done every 5s.
Before all connections are recycled, the second reload @@config_all will throw error.

@actiontech-bot
Copy link
Member Author

By @yanhuqing666

Atomic is not guaranteed yet.

@zhunina
Copy link

zhunina commented Sep 8, 2017

fixed

@zhunina zhunina closed this as completed Sep 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants