Michael Widenius <michael.widenius@gmail.com> writes:
What do you think about changing this to use the async deadlock kill in background thread, as discussed in this thread?
https://lists.launchpad.net/maria-developers/msg09902.html
This would allow to simplify the code in lock0lock.cc, and avoid the locking hacks in innobase_kill_query()?
I was trying to find the exact patch/patches you are referring to.
https://github.com/knielsen/server/commit/841ada8c8ac39c024cd1eafe4b346deecb...
https://github.com/knielsen/server/commit/b256733df2cf9f10d38e44ca4979843a3b...
Is it possible for you to create a clean patch for async deadlock for bb-10.2-jan that Jan and I can review and apply?
Yes, I'll prepare a patch for review. It's probably easier if I apply it myself, as I want to do a minimal patch in 10.1 to get optimistic parallel replication working with TokuDB; and a full patch in 10.2 which includes the InnoDB simplifications. - Kristian.