metrics should listen to MetricsEnabled not TraceEnabled cc @crunchie84

Author: Mpdreamz

src/Elasticsearch.Net/Connection/RequestState/TransportRequestState.cs

@@ -13,6 +13,8 @@ namespace Elasticsearch.Net.Connection.RequestState
 	public class TransportRequestState<T> : IDisposable, ITransportRequestState
 	{
 		private readonly bool _traceEnabled;
+		private readonly bool _metricsEnabled;
+
 		private Stopwatch _stopwatch;
 
 		private ElasticsearchResponse<T> _result;
@@ -91,8 +93,9 @@ public TransportRequestState(
 			this.ClientSettings = settings;
 			this.RequestParameters = requestParameters;
 			this._traceEnabled = settings.TraceEnabled;
-			if (this._traceEnabled)
-				this._stopwatch = Stopwatch.StartNew(); this.Method = method;
+			this._metricsEnabled = settings.MetricsEnabled;
+			if (this._metricsEnabled)
+				this._stopwatch = Stopwatch.StartNew(); 
 
 			this.Method = method;
 			this.Path = path;
@@ -107,7 +110,7 @@ public TransportRequestState(
 		public void TickSerialization(byte[] postData)
 		{
 			this.PostData = postData;
-			if (this._traceEnabled)
+			if (this._metricsEnabled)
 				this.SerializationTime = this._stopwatch.ElapsedMilliseconds;
 		}
 
@@ -139,8 +142,7 @@ public void SetResult(ElasticsearchResponse<T> result)
 		{
 			if (result == null)
 			{
-				if (!_traceEnabled) return;
-				this._stopwatch.Stop();
+				if (this._stopwatch != null) this._stopwatch.Stop();
 				return;
 			}
 			result.NumberOfRetries = this.Retried;
@@ -160,11 +162,12 @@ public void SetResult(ElasticsearchResponse<T> result)
 
 			if (this.ClientSettings.ConnectionStatusHandler != null)
 				this.ClientSettings.ConnectionStatusHandler(result);
+			
+			if (this._stopwatch != null) this._stopwatch.Stop();
 
 			if (!_traceEnabled) return;
 
 			this._result = result;
-			this._stopwatch.Stop();
 
 		}