Don't call shutdown on connection failures
Don't call shutdown on connection failures

Calling shutdown causes data races since the reader/writer goroutine may
still have a reference to the sendCh, but shutdown decrements this
reference. We should instead unblock readers/writers by notifying them
of the error, and they can call shutdown.

uber/tchannel-go,uber/tchannel-go