AbstractClientChannel should not swallow inner exceptions

Nifty swallows inner exceptions and rethrows TTransportException with "Client was disconnected by server", which is misleading and it causes debugging to be really hard. This is to switch the order of the calls so that we can keep the original exception for better debugging.