remove unnecessary flag, use async call for ping

Author: Dmitry-Bryliuk

src/tarantool.client/LogicalConnectionManager.cs

@@ -17,12 +17,10 @@ public class LogicalConnectionManager : ILogicalConnection
 
         private LogicalConnection _droppableLogicalConnection;
 
-        private readonly ManualResetEvent _connected = new ManualResetEvent(false);
+        private readonly ManualResetEvent _connected = new ManualResetEvent(true);
 
         private readonly AutoResetEvent _reconnectAvailable = new AutoResetEvent(true);
 
-        private bool _onceConnected;
-
         private Timer _timer;
 
         private int _disposing;
@@ -77,8 +75,6 @@ public async Task Connect()
 
                 _connected.Reset();
 
-                _onceConnected = true;
-
                 _clientOptions.LogWriter?.WriteLine($"{nameof(LogicalConnectionManager)}: Connecting...");
 
                 var _newConnection = new LogicalConnection(_clientOptions, _requestIdCounter);
@@ -111,7 +107,10 @@ private void CheckPing()
                     return;
                 }
 
-                Task.WaitAny(SendRequestWithEmptyResponse(_pingRequest));
+                // suppress for fire and forget behaviour
+#pragma warning disable 4014
+                SendRequestWithEmptyResponse(_pingRequest);
+#pragma warning restore 4014
             }
             finally
             {
@@ -124,7 +123,7 @@ private void CheckPing()
 
         public bool IsConnected()
         {
-            if (!_onceConnected || !_connected.WaitOne(_connectionTimeout))
+            if (!_connected.WaitOne(_connectionTimeout))
             {
                 return false;
             }