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.

Sign up for GitHub

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

Jump to bottom

Update torndb.py #13

Open

learsu wants to merge 2 commits into bdarnell:master from learsu:patch-1

learsu commented Jul 31, 2013

support transaction,no false reason,if you want reason,del L149 #

learsu added 2 commits

July 31, 2013 16:34


          Update torndb.py

78109cf

support transaction,no false reason,if you want reason,del L149 #


          Update torndb.py

a3a7b95

method _ensure_connected add ping check connect

felix021 commented Sep 11, 2013

The reconnect() call you added may conflict will the transaction feature.

The comments in libmysql/client.c says:

By default we don't reconnect because it could silently corrupt data (after reconnection you potentially lose table locks, user variables, session variables (transactions but they are specifically dealt with in mysql_reconnect()). This is a change: < 5.0.3 mysql->reconnect was set to 1 by default.

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

Labels

None yet