Release connection before completing the future in RequestHandler#onSet
As part of the fix for JAVA-633, the release of the connection in RequestHandler#onSet was moved to a finally block (because we want to hold onto the connection in the case where we get an UNPREPARED response).
This was a bad idea because now the connection is not released until after the future has been set. If a client uses Future.transform without a separate executor, and their callback take a long time to execute (like executing another request synchronously), we end up holding on the connection way too long. See this discussion.
Refactor the code to release the connection first, like it was done before (will just need a bit of duplication to avoid doing it in the UNPREPARED case).
This affects the same code as JAVA-561, so I'll do it on the same branch to avoid extra merge work.
I'll note that changing the connection releasing is not a bad idea, the shorter we hold a connection the better, but as I commented in the linked discussion I don't really think it matters much for the case you describe in the sense that if you use a callback without a separate executor, your callback should not take a long time to execute (or you'll slow down the whole driver), and even less execute another request synchronously (or you'll deadlock).