Fix in "upstream" not "synced" to this repo

[walro]

We recently switched over to using your fork rather than the original version, as a result of that I checked the difference between original/"upstream" and this fork, using: master...shyiko:master

It appears that the fix for shyiko#321 is not included in this repo, as it was commited after "time-of-fork". In particular it's the following two commits that could potentially be of use here too: shyiko/mysql-binlog-connector-java@0.20.1...c54b4c1

Cherry-picking the first commit, e59006e, worked without any hassle. Things got a bit hairy when trying to apply c54b4c1 as lines in the same file were touched later on.

I will try to put up a PR, but preemptively created this issue in case I fail :-)

Thanks for keeping the project alive!

[osheroff]

ugh, yeah. c54b4c1 has a lot of refactoring / unrelated changes that are deeply in conflict. do you think we could find a more surgical fix for whatever the deadlock issue there was?

[walro]

Looking at the latter commit, ignoring white-space changes, one ends up with: c54b4c1?w=1 which is a bit more manageable perhaps. I'll assume that the SSL stuff is unrelated. Also I imagine one could live without the renaming of the ensureEventDataDeserialize method.

I'll see if I can can even get a re-pro going for starters, as no tests were added. (Internally, we actually run without keepAlive set to true, so we are probably very unlikely to ever run into this problem)