Merge pull request #624 from danp60/bugfix/close_bad_thrift_connection

Mpdreamz · Mpdreamz · commit 43a14149acad · 2014-04-27T12:20:02.000+02:00
Also close Thrift connection on TTransportException
diff --git a/src/Connections/Elasticsearch.Net.Connection.Thrift/ThriftConnection.cs b/src/Connections/Elasticsearch.Net.Connection.Thrift/ThriftConnection.cs
@@ -323,6 +323,11 @@ private ElasticsearchResponse<Stream> Execute(RestRequest restRequest, object de
 					client.InputProtocol.Transport.Close();
 					throw;
 				}
+				catch (TTransportException)
+				{
+					client.InputProtocol.Transport.Close();
+					throw;
+				}
 				finally
 				{
 					//make sure we make the client available again.

Original file line number	Diff line number	Diff line change
`@@ -323,6 +323,11 @@ private ElasticsearchResponse<Stream> Execute(RestRequest restRequest, object de`
`323`	`323`	`client.InputProtocol.Transport.Close();`
`324`	`324`	`throw;`
`325`	`325`	`}`
	`326`	`+ catch (TTransportException)`
	`327`	`+ {`
	`328`	`+ client.InputProtocol.Transport.Close();`
	`329`	`+ throw;`
	`330`	`+ }`
`326`	`331`	`finally`
`327`	`332`	`{`
`328`	`333`	`//make sure we make the client available again.`